Cysql bench optimizer

[zinic]

Description

Resolves: <TICKET_OR_ISSUE_NUMBER>

Type of Change

• Chore (a change that does not modify the application functionality)
• Bug fix (a change that fixes an issue)
• New feature / enhancement (a change that adds new functionality)
• Refactor (no behaviour change)
• Test coverage
• Build / CI / tooling
• Documentation

Testing

• Unit tests added / updated
• Integration tests added / updated
• Full test suite run (make test_all with CONNECTION_STRING set)

Screenshots (if appropriate):

Driver Impact

• PostgreSQL driver (drivers/pg)
• Neo4j driver (drivers/neo4j)

Checklist

• Code is formatted
• All existing tests pass
• go.mod / go.sum are up to date if dependencies changed

[coderabbitai]

Important

Review skipped

Too many files!

This PR contains 207 files, which is 57 over the limit of 150.

To get a review, reduce the PR to 150 files or fewer by splitting it into smaller PRs or changing its base branch.

Upgrade to Pro+ to raise the limit.

This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: de53d7d3-623c-4c59-aec5-16a36173078d

📥 Commits

Reviewing files that changed from the base of the PR and between 119ca13 and b50764e.

📒 Files selected for processing (207)

• .github/workflows/go-test.yml
• Makefile
• README.md
• artifacts/perf/production-lift-final/REPORT.md
• artifacts/perf/real-world-live-v2/anchors.json
• artifacts/perf/real-world-live-v2/compile.jsonl
• artifacts/perf/real-world-live-v2/concurrency.jsonl
• artifacts/perf/real-world-live-v2/dataset.json
• artifacts/perf/real-world-live-v2/harness.go.txt
• artifacts/perf/real-world-live-v2/harness_test.go.txt
• artifacts/perf/real-world-live-v2/pilot-edge-cases.jsonl
• artifacts/perf/real-world-live-v2/plans.jsonl
• artifacts/perf/real-world-live-v2/results.jsonl
• artifacts/perf/real-world-live/REPORT.md
• artifacts/perf/real-world-live/dataset.json
• artifacts/perf/real-world-live/harness.go.txt
• artifacts/perf/real-world-live/postgres-plans.jsonl
• artifacts/perf/real-world-live/postgres-results.jsonl
• benchmark/testdata/scale/README.md
• benchmark/testdata/scale/cases/generated_adcs.json
• benchmark/testdata/scale/cases/generated_shortest_paths.json
• benchmark/testdata/scale/cases/hops.json
• benchmark/testdata/scale/cases/reconciliation.json
• benchmark/testdata/scale/cases/scans_lookups.json
• benchmark/testdata/scale/cases/shortest_paths.json
• benchmark/testdata/scale/cases/traversal.json
• benchmark/testdata/scale/cases/trust_pruning.json
• cmd/graphbench/README.md
• cmd/graphbench/aa_report.go
• cmd/graphbench/aa_report_test.go
• cmd/graphbench/bundle.go
• cmd/graphbench/concurrency.go
• cmd/graphbench/concurrency_test.go
• cmd/graphbench/confirm_report.go
• cmd/graphbench/confirm_report_test.go
• cmd/graphbench/corpus.go
• cmd/graphbench/corpus_test.go
• cmd/graphbench/datasets.go
• cmd/graphbench/datasets_test.go
• cmd/graphbench/dormant_forms_guard_test.go
• cmd/graphbench/environment.go
• cmd/graphbench/environment_test.go
• cmd/graphbench/main.go
• cmd/graphbench/main_test.go
• cmd/graphbench/measure.go
• cmd/graphbench/measure_test.go
• cmd/graphbench/neo4j.go
• cmd/graphbench/perf_gate.go
• cmd/graphbench/perf_gate_test.go
• cmd/graphbench/postgres.go
• cmd/graphbench/postgres_plan.go
• cmd/graphbench/postgres_plan_test.go
• cmd/graphbench/postgres_test.go
• cmd/graphbench/postgresql_plan_invariants_integration_test.go
• cmd/graphbench/reference_closure_report.go
• cmd/graphbench/reference_closure_report_test.go
• cmd/graphbench/reference_pair_report.go
• cmd/graphbench/reference_pair_report_test.go
• cmd/graphbench/references.go
• cmd/graphbench/references_test.go
• cmd/graphbench/results.go
• cmd/graphbench/results_test.go
• cmd/graphbench/run_lock.go
• cmd/graphbench/run_lock_test.go
• cmd/graphbench/scale_corpus_contract_test.go
• cmd/graphbench/selection.go
• cmd/graphbench/summary.go
• cmd/graphbench/summary_test.go
• cmd/graphbench/types.go
• cmd/graphbench/waterfall.go
• cmd/graphbench/waterfall_test.go
• cmd/plancorpus/README.md
• cmd/plancorpus/capture.go
• cmd/plancorpus/corpus.go
• cmd/plancorpus/corpus_test.go
• cmd/plancorpus/dormant_forms_guard_test.go
• cmd/plancorpus/main.go
• cmd/plancorpus/main_test.go
• cmd/plancorpus/report.go
• cmd/plancorpus/types.go
• cypher/models/cypher/functions.go
• cypher/models/pgsql/format/format.go
• cypher/models/pgsql/format/format_test.go
• cypher/models/pgsql/functions.go
• cypher/models/pgsql/model.go
• cypher/models/pgsql/optimize/lowering.go
• cypher/models/pgsql/optimize/lowering_plan.go
• cypher/models/pgsql/optimize/optimizer_test.go
• cypher/models/pgsql/optimize/scalar_continuation_test.go
• cypher/models/pgsql/optimize/source_references.go
• cypher/models/pgsql/test/logical_forms_legacy_builder_test.go
• cypher/models/pgsql/test/reconciliation_forms_legacy_builder_test.go
• cypher/models/pgsql/test/relationship_scans_node_lookups_legacy_builder_test.go
• cypher/models/pgsql/test/standalone_hop_forms_legacy_builder_test.go
• cypher/models/pgsql/test/testcase.go
• cypher/models/pgsql/test/translation_cases/delete.sql
• cypher/models/pgsql/test/translation_cases/multipart.sql
• cypher/models/pgsql/test/translation_cases/nodes.sql
• cypher/models/pgsql/test/translation_cases/pattern_binding.sql
• cypher/models/pgsql/test/translation_cases/pattern_expansion.sql
• cypher/models/pgsql/test/translation_cases/post_processing.sql
• cypher/models/pgsql/test/translation_cases/quantifiers.sql
• cypher/models/pgsql/test/translation_cases/reconciliation.sql
• cypher/models/pgsql/test/translation_cases/relationship_scans_node_lookups.sql
• cypher/models/pgsql/test/translation_cases/scalar_aggregation.sql
• cypher/models/pgsql/test/translation_cases/shortest_paths.sql
• cypher/models/pgsql/test/translation_cases/stepwise_traversal.sql
• cypher/models/pgsql/test/translation_cases/unwind.sql
• cypher/models/pgsql/test/translation_cases/update.sql
• cypher/models/pgsql/test/translation_test.go
• cypher/models/pgsql/test/trust_pruning_forms_legacy_builder_test.go
• cypher/models/pgsql/translate/adcs_suffix_seeded.go
• cypher/models/pgsql/translate/expansion.go
• cypher/models/pgsql/translate/expansion_test.go
• cypher/models/pgsql/translate/expression.go
• cypher/models/pgsql/translate/expression_test.go
• cypher/models/pgsql/translate/format.go
• cypher/models/pgsql/translate/function.go
• cypher/models/pgsql/translate/function_test.go
• cypher/models/pgsql/translate/graph_scope_test.go
• cypher/models/pgsql/translate/hinting.go
• cypher/models/pgsql/translate/limit_pushdown_test.go
• cypher/models/pgsql/translate/model.go
• cypher/models/pgsql/translate/optimizer_safety_test.go
• cypher/models/pgsql/translate/path_functions.go
• cypher/models/pgsql/translate/pattern.go
• cypher/models/pgsql/translate/projection.go
• cypher/models/pgsql/translate/renamer.go
• cypher/models/pgsql/translate/shortest_workspace_test.go
• cypher/models/pgsql/translate/tracking.go
• cypher/models/pgsql/translate/translator.go
• cypher/models/pgsql/translate/traversal.go
• cypher/models/walk/walk_pgsql.go
• cypher/test/cases/mutation_tests.json
• cypher/test/cases/positive_tests.json
• cypher/test/test.go
• docs/development.md
• docs/performance_l3a_discovery.md
• docs/performance_l3m_m0_qualification.md
• docs/performance_plan_completion.md
• docs/postgresql_translation.md
• docs/regression_source_parity.md
• drivers/pg/batch.go
• drivers/pg/batch_test.go
• drivers/pg/composite_codec.go
• drivers/pg/composite_codec_integration_test.go
• drivers/pg/composite_codec_test.go
• drivers/pg/driver.go
• drivers/pg/manager.go
• drivers/pg/mapper.go
• drivers/pg/mapper_test.go
• drivers/pg/pg.go
• drivers/pg/query/sql/schema_down.sql
• drivers/pg/query/sql/schema_up.sql
• drivers/pg/query/sql_workspace_test.go
• drivers/pg/query_cache.go
• drivers/pg/query_cache_test.go
• drivers/pg/result.go
• drivers/pg/result_test.go
• drivers/pg/transaction.go
• drivers/pg/types.go
• integration/cypher_template_test.go
• integration/cypher_test.go
• integration/delegated_enrollment_legacy_builder_test.go
• integration/direct_write_mutations_test.go
• integration/harness.go
• integration/legacy_query_harness.go
• integration/logical_forms_legacy_builder_test.go
• integration/regression_fixture.go
• integration/regression_fixture_test.go
• integration/relationship_scans_node_lookups_legacy_builder_test.go
• integration/standalone_hops_legacy_builder_test.go
• integration/testdata/README.md
• integration/testdata/cases/mutation_post_state_inline.json
• integration/testdata/cases/shortest_bound.json
• integration/testdata/shortest_bound.json
• integration/testdata/templates/advanced_lookup_shapes.json
• integration/testdata/templates/basic_lookup_shapes.json
• integration/testdata/templates/count_shapes.json
• integration/testdata/templates/mutation_post_state_shapes.json
• integration/testdata/templates/post_processing_hop_shapes.json
• integration/testdata/templates/post_processing_shapes.json
• integration/testdata/templates/reconciliation_shapes.json
• integration/testdata/templates/relationship_scan_shapes.json
• integration/trust_pruning_legacy_builder_test.go
• integration/wipe_graph_test.go
• perf_cont_1.md
• perf_cont_2.md
• perf_cont_3.md
• perf_cont_4.md
• perf_rework_plan.md
• query/neo4j/neo4j_test.go
• query/neo4j/relationship_scans_node_lookups_test.go
• query/neo4j/rewrite.go
• query/v2/backend_test.go
• regression_coverage_manifest.md
• regression_manifest_test.go
• regression_plan.md
• testutil/metadata.go
• testutil/metadata_test.go
• testutil/params.go
• testutil/params_test.go
• testutil/perf_fixtures.go
• testutil/perf_fixtures_test.go
• testutil/reconciliation_fixture.go
• testutil/reconciliation_fixture_test.go
• tools/dawgrun/pkg/commands/cypher.go

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

_{Comment @coderabbitai help to get the list of available commands.}