Skip to content

BE-455: HashQL: Introduce evaluation orchestrator#8586

Draft
indietyp wants to merge 67 commits intobm/be-306-hashql-postgres-translationfrom
bm/be-455-hashql-create-evaluation-target-coordinator
Draft

BE-455: HashQL: Introduce evaluation orchestrator#8586
indietyp wants to merge 67 commits intobm/be-306-hashql-postgres-translationfrom
bm/be-455-hashql-create-evaluation-target-coordinator

Conversation

@indietyp
Copy link
Member

🌟 What is the purpose of this PR?

🔗 Related links

  • ...

🚫 Blocked by

  • ...

🔍 What does this change?

  • ...

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

  • are internal and do not require a docs change

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

  • do not affect the execution graph

⚠️ Known issues

🐾 Next steps

🛡 What tests cover this?

❓ How to test this?

  1. Checkout the branch / view the deployment
  2. Try X
  3. Confirm that Y

📹 Demo

@vercel
Copy link

vercel bot commented Mar 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hash Ready Ready Preview, Comment Mar 26, 2026 8:29am
petrinaut Ready Ready Preview Mar 26, 2026 8:29am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
hashdotdesign Ignored Ignored Preview Mar 26, 2026 8:29am
hashdotdesign-tokens Ignored Ignored Preview Mar 26, 2026 8:29am

@codecov
Copy link

codecov bot commented Mar 26, 2026

Codecov Report

❌ Patch coverage is 62.91310% with 303 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.45%. Comparing base (c015a34) to head (95af2dc).

Files with missing lines Patch % Lines
libs/@local/hashql/compiletest/src/pipeline.rs 0.00% 123 Missing ⚠️
libs/@local/hashql/core/src/heap/pool.rs 29.67% 64 Missing ⚠️
...src/store/postgres/query/expression/conditional.rs 0.00% 41 Missing ⚠️
...src/module/std_lib/graph/types/knowledge/entity.rs 87.07% 34 Missing ⚠️
...l/hashql/core/src/module/std_lib/graph/temporal.rs 77.85% 31 Missing ⚠️
...ore/src/module/std_lib/graph/types/ontology/mod.rs 85.71% 5 Missing ⚠️
...e/std_lib/graph/types/principal/actor_group/web.rs 81.25% 3 Missing ⚠️
...e/std_lib/graph/types/principal/actor_group/mod.rs 87.50% 2 Missing ⚠️
Additional details and impacted files
@@                            Coverage Diff                             @@
##           bm/be-306-hashql-postgres-translation    #8586       +/-   ##
==========================================================================
- Coverage                                  63.93%   53.45%   -10.49%     
==========================================================================
  Files                                       1195      976      -219     
  Lines                                     130685    94048    -36637     
  Branches                                    5005     4035      -970     
==========================================================================
- Hits                                       83555    50272    -33283     
+ Misses                                     46263    43239     -3024     
+ Partials                                     867      537      -330     
Flag Coverage Δ
apps.hash-ai-worker-ts 1.40% <ø> (ø)
apps.hash-api 0.00% <ø> (ø)
blockprotocol.type-system 40.84% <ø> (ø)
local.claude-hooks 0.00% <ø> (ø)
local.harpc-client 51.24% <ø> (ø)
local.hash-graph-sdk 9.63% <ø> (ø)
local.hash-isomorphic-utils 0.00% <ø> (ø)
rust.antsi 0.00% <ø> (ø)
rust.error-stack 90.88% <ø> (ø)
rust.harpc-codec 84.70% <ø> (ø)
rust.harpc-net 96.16% <ø> (ø)
rust.harpc-tower 66.80% <ø> (ø)
rust.harpc-types 0.00% <ø> (ø)
rust.harpc-wire-protocol 92.23% <ø> (ø)
rust.hash-codec 72.76% <ø> (ø)
rust.hash-graph-api 2.52% <ø> (ø)
rust.hash-graph-authorization 62.34% <ø> (ø)
rust.hash-graph-postgres-store 26.74% <0.00%> (-0.07%) ⬇️
rust.hash-graph-store 37.76% <ø> (ø)
rust.hash-graph-temporal-versioning 47.95% <ø> (ø)
rust.hash-graph-types 0.00% <ø> (ø)
rust.hash-graph-validation 83.45% <ø> (ø)
rust.hashql-ast ?
rust.hashql-compiletest 28.34% <0.00%> (-1.36%) ⬇️
rust.hashql-core 82.17% <78.51%> (-0.27%) ⬇️
rust.hashql-diagnostics 72.53% <100.00%> (+0.09%) ⬆️
rust.hashql-eval ?
rust.hashql-hir ?
rust.hashql-mir ?
rust.hashql-syntax-jexpr 94.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 26, 2026

Merging this PR will degrade performance by 17.99%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 1 regressed benchmark
✅ 79 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
unique[100] 20.2 µs 24.6 µs -17.99%

Comparing bm/be-455-hashql-create-evaluation-target-coordinator (95af2dc) with bm/be-306-hashql-postgres-translation (c015a34)

Open in CodSpeed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/deps Relates to third-party dependencies (area) area/infra Relates to version control, CI, CD or IaC (area) area/libs Relates to first-party libraries/crates/packages (area) area/tests New or updated tests type/eng > backend Owned by the @backend team

Development

Successfully merging this pull request may close these issues.

1 participant