fdb-record-layer icon indicating copy to clipboard operation
fdb-record-layer copied to clipboard

Fixes #2573: Delete PrimaryKeySegmentIndex entries when files are deleted

Open ScottDugas opened this issue 1 year ago • 9 comments

Since I new this would involve a lot of extracting of interfaces, and moving code I thought it might be easier to go commit-by-commit, and have some pretty simple commits for that kind of stuff. Some of the merges a little later on didn't go super smoothly, so it might be better to diff the 3 versions of: LucenePrimaryKeySegmentIndex (v1, v2 and the interface).

I was unsure whether to take Lucene's approach, and copy things to avoid breaking things, or extract base classes. I made two different decisions here. Looking back it may make sense to have LucenePrimaryKeySegmentIndexV1 extend from LucenePrimaryKeySegmentIndexV2 based on implementations (perhaps better names would make that make more sense). However, given the corruption, and the current state of lucene, I expect V1 to be removed shortly, so it probably doesn't matter.

ScottDugas avatar Mar 13 '24 21:03 ScottDugas

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: 67585da4e26df5a75bea99d69014aed76599041e
  • Duration 0:40:14
  • Result: :x: FAILED
  • Error: Error while executing command: ./gradlew --no-daemon --console=plain -b ./build.gradle build destructiveTest -PcoreNotStrict -PreleaseBuild=false -PpublishBuild=false -PspotbugsEnableHtmlReport. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 13 '24 21:03 foundationdb-ci

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: ba3c1da47e59302f23eee0138dbe28e7c1f39444
  • Duration 0:43:29
  • Result: :x: FAILED
  • Error: Error while executing command: ./gradlew --no-daemon --console=plain -b ./build.gradle build destructiveTest -PcoreNotStrict -PreleaseBuild=false -PpublishBuild=false -PspotbugsEnableHtmlReport. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 14 '24 19:03 foundationdb-ci

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: ab4d170d7f04d843ebb6cd07e3f41eb81baed9ec
  • Duration 0:47:55
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 14 '24 19:03 foundationdb-ci

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: 24568bdf3d80822d86d0c9b0fd759fd66a0a14f8
  • Duration 0:47:18
  • Result: :x: FAILED
  • Error: Error while executing command: ./gradlew --no-daemon --console=plain -b ./build.gradle build destructiveTest -PcoreNotStrict -PreleaseBuild=false -PpublishBuild=false -PspotbugsEnableHtmlReport. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 14 '24 21:03 foundationdb-ci

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: b8274aac874f4713a5e81203dc0a7634465638fe
  • Duration 0:46:08
  • Result: :x: FAILED
  • Error: Error while executing command: ./gradlew --no-daemon --console=plain -b ./build.gradle build destructiveTest -PcoreNotStrict -PreleaseBuild=false -PpublishBuild=false -PspotbugsEnableHtmlReport. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 15 '24 16:03 foundationdb-ci

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: 9f9891f8b427150e0c38e9725e4fcd51e115a968
  • Duration 0:45:37
  • Result: :x: FAILED
  • Error: Error while executing command: ./gradlew --no-daemon --console=plain -b ./build.gradle build destructiveTest -PcoreNotStrict -PreleaseBuild=false -PpublishBuild=false -PspotbugsEnableHtmlReport. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 15 '24 19:03 foundationdb-ci

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: 92f8780d11e5a3876f9d96ea439022edf47bf410
  • Duration 0:45:35
  • Result: :x: FAILED
  • Error: Error while executing command: ./gradlew --no-daemon --console=plain -b ./build.gradle build destructiveTest -PcoreNotStrict -PreleaseBuild=false -PpublishBuild=false -PspotbugsEnableHtmlReport. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 15 '24 22:03 foundationdb-ci

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: da6a112b9b759a772e25e0fafcb96c26db0fb1ca
  • Duration 0:46:25
  • Result: :x: FAILED
  • Error: Error while executing command: ./gradlew --no-daemon --console=plain -b ./build.gradle build destructiveTest -PcoreNotStrict -PreleaseBuild=false -PpublishBuild=false -PspotbugsEnableHtmlReport. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 16 '24 15:03 foundationdb-ci

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: fe36f0781ea65e3383013951e5986c84940302f1
  • Duration 0:47:01
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 16 '24 21:03 foundationdb-ci

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: f9987e5b3bafcf9c21f88ef04159e6e2175ad774
  • Duration 0:45:45
  • Result: :x: FAILED
  • Error: Error while executing command: ./gradlew --no-daemon --console=plain -b ./build.gradle build destructiveTest -PcoreNotStrict -PreleaseBuild=false -PpublishBuild=false -PspotbugsEnableHtmlReport. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 18 '24 22:03 foundationdb-ci

Result of fdb-record-layer-pr on Linux CentOS 7

  • Commit ID: 36f2d28a597eb8575501a35352b4c140c934157d
  • Duration 0:47:08
  • Result: :white_check_mark: SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci avatar Mar 19 '24 03:03 foundationdb-ci