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

Published 20 hours ago • verified publisher icon FoundationDB

Resolves #2801: Covering optimization with invertible index fields

Open MMcM opened this issue 1 year ago • 5 comments

This is complete for the heuristic planner. For Cascades, it adds the necessary partial record support, but more work in tryFetchCoveringIndexScan is needed, depending on how precisely function index keys are represented in Values.

MMcM avatar Jul 03 '24 23:07 MMcM

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

  • Commit ID: 01d24d96db515d8432e029dafa514dd7ed896517
  • Duration 0:15:23
  • 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 Jul 03 '24 23:07 foundationdb-ci

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

  • Commit ID: 575b56803aa6106a6bc30c00bb778ca7bca77f73
  • Duration 0:16:06
  • 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 Jul 04 '24 00:07 foundationdb-ci

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

  • Commit ID: 92547815f4a461657c533a14cd867c5e2e49f840
  • Duration 0:36:10
  • 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 Jul 04 '24 03:07 foundationdb-ci

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

  • Commit ID: 967636d5968f89141e18d101f8d7a897a96d5425
  • Duration 0:37:04
  • 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 Jul 04 '24 05:07 foundationdb-ci

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

  • Commit ID: 50db9a781279ea134e117daaf0eec3fe49b0f9bc
  • Duration 0:46:27
  • 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 Jul 04 '24 06:07 foundationdb-ci

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

  • Commit ID: f689f2ef04ffe93cf4a0ceee8bdfc4ae35292e25
  • Duration 0:45:46
  • 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 Jul 04 '24 15:07 foundationdb-ci

I guess there's a question as to whether we should enhance FieldData to allow for more general computation. I think given that we want to replace it with Cascades primitives anyway (which should already be able to handle that sort of them), having the extra field on the copier is fine for the old planner

alecgrieser avatar Jul 05 '24 15:07 alecgrieser