Skip to content

feat: support AND, OR, NOT and NULL in cross-shard HAVING - #1908

Open
murex971 wants to merge 3 commits into
murex-having-supportfrom
having-boolean
Open

murex971 wants to merge 3 commits into
murex-having-supportfrom
having-boolean

Conversation

@murex971

Copy link
Copy Markdown
Contributor

No description provided.

@murex971
murex971 added this pull request to stack #1907 October 27, 2026 16:28
@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

🐰 Bencher Report

ProjectPgDog
Branchhaving-boolean
TestbedIntel v1
Click to view all benchmark results
BenchmarkThroughputBenchmark Result
operations / second (ops/s) x 1e3
(Result Δ%)
Lower Boundary
operations / second (ops/s) x 1e3
(Limit %)
cpu-per-txnBenchmark Result
Measure (units)
(Result Δ%)
Upper Boundary
Measure (units)
(Limit %)
pooler-v3📈 view plot
🚷 view threshold
16.99 ops/s x 1e3
(+0.00%)Baseline: 16.99 ops/s x 1e3
16.82 ops/s x 1e3
(99.00%)
📈 view plot
🚷 view threshold
40.96 units
(+0.48%)Baseline: 40.77 units
41.17 units
(99.49%)
🐰 View full continuous benchmarking report in Bencher

@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

🐰 Bencher Report

ProjectPgDog
Branchhaving-boolean
TestbedIntel v1
Click to view all benchmark results
BenchmarkThroughputBenchmark Result
operations / second (ops/s) x 1e3
(Result Δ%)
Lower Boundary
operations / second (ops/s) x 1e3
(Limit %)
cpu-per-txnBenchmark Result
Measure (units)
(Result Δ%)
Upper Boundary
Measure (units)
(Limit %)
lb-v3📈 view plot
🚷 view threshold
15.99 ops/s x 1e3
(-0.00%)Baseline: 15.99 ops/s x 1e3
15.83 ops/s x 1e3
(99.00%)
📈 view plot
🚷 view threshold
42.18 units
(+0.81%)Baseline: 41.84 units
42.26 units
(99.81%)
🐰 View full continuous benchmarking report in Bencher

@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

🐰 Bencher Report

ProjectPgDog
Branchhaving-boolean
TestbedIntel v1
Click to view all benchmark results
BenchmarkThroughputBenchmark Result
operations / second (ops/s) x 1e3
(Result Δ%)
Lower Boundary
operations / second (ops/s) x 1e3
(Limit %)
cpu-per-txnBenchmark Result
Measure (units)
(Result Δ%)
Upper Boundary
Measure (units)
(Limit %)
ctx-switches-per-txnBenchmark Result
Measure (units)
(Result Δ%)
Upper Boundary
Measure (units)
(Limit %)
sharding-w2-v3📈 view plot
🚷 view threshold
10.00 ops/s x 1e3
(+0.00%)Baseline: 10.00 ops/s x 1e3
9.90 ops/s x 1e3
(99.00%)
📈 view plot
🚷 view threshold
62.95 units
(+0.31%)Baseline: 62.75 units
63.38 units
(99.32%)
📈 view plot
🚷 view threshold
1.26 units
(-0.13%)Baseline: 1.26 units
1.29 units
(96.97%)
🐰 View full continuous benchmarking report in Bencher

@github-actions

Copy link
Copy Markdown
Contributor

Metal bench failed. 1 of 150 measures outside bounds.

suite throughput cpu/txn user cycles/txn user instr/txn kernel cycles/txn ctx/txn
pooler 28.58 (-0.3% ± 0.8%) 28,211 (-0.6% ± 1.0%) 35,215 (-0.3% ± 0.02%) 39,587 (-0.3% ± 1.1%)
lb 29.53 (-0.7% ± 1.1%) 29,734 (-0.5% ± 1.0%) 36,073 (-0.3% ± 0.03%) 40,434 (-1.0% ± 1.9%)
sharding-w2 52.91 (+0.8%) 67,101 (+0.0%) 72,441 (-0.2%) 61,382 (+1.8%) 1.413 (-0.2%)
sharding-w8 158,524 (-1.5%) 46.06 (+0.8%) 65,923 (-0.9%) 56,126 (-0.2%) 52,668 (+3.1%) 0.1479 (+4.4%), bound +3%
Counters per txn
suite frontend stall l1i refill backend stall
pooler 14,254 (+0.3% ± 1.9%) 3,242 (+0.3% ± 0.9%) 2,573 (-4.4% ± 1.5%)
lb 15,221 (+0.2% ± 2.2%) 3,327 (-0.1% ± 1.1%) 2,814 (-3.6% ± 2.1%)
sharding-w2 37,918 (+0.4%) 5,347 (-0.5%) 7,186 (+1.9%)
sharding-w8 35,807 (-4.1%) 4,967 (-0.9%) 10,244 (+9.6%)

Microbench: all 135 measures within bounds.

  • extended_cached: 15 of 15 benches faster, median -3.6% ± 0.2%
  • extended_cold: 13 of 15 benches faster, median -2.0% ± 0.6%
  • simple_cold: 15 of 15 benches faster, median -3.0% ± 0.3%
lb flamegraph vs d74cf08

lb

Open interactive for hover, zoom and search.

lb removed

Open interactive.

microbench flamegraph vs d74cf08

microbench

Open interactive for hover, zoom and search.

microbench removed

Open interactive.

pooler flamegraph vs d74cf08

pooler

Open interactive for hover, zoom and search.

pooler removed

Open interactive.

sharding-w2 flamegraph vs d74cf08

sharding-w2

Open interactive for hover, zoom and search.

sharding-w2 removed

Open interactive.

sharding-w8 flamegraph vs d74cf08

sharding-w8

Open interactive for hover, zoom and search.

sharding-w8 removed

Open interactive.

Logs

lb, microbench, pooler, sharding-w2, sharding-w8, verdict.json

Compared 1f087da with d74cf08.

@codecov

codecov Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.96403% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...frontend/router/parser/rewrite/statement/having.rs 88.52% 7 Missing ⚠️

📢 Thoughts on this report? Let us know!

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant