FoundationDB / fdb-record-layer

A record-oriented store built on FoundationDB
Apache License 2.0
570 stars 102 forks source link

Resolves #2801: Covering optimization with invertible index fields #2802

Closed MMcM closed 1 week ago

MMcM commented 1 week ago

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.

foundationdb-ci commented 1 week ago

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

foundationdb-ci commented 1 week ago

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

foundationdb-ci commented 1 week ago

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

foundationdb-ci commented 1 week ago

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

foundationdb-ci commented 1 week ago

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

foundationdb-ci commented 1 week ago

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

alecgrieser commented 1 week ago

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