Skip to content

semi/anti join output statistics#171

Open
korowa wants to merge 2 commits intomasterfrom
semijoin_statistics
Open

semi/anti join output statistics#171
korowa wants to merge 2 commits intomasterfrom
semijoin_statistics

Conversation

@korowa
Copy link
Owner

@korowa korowa commented Mar 26, 2024

Which issue does this PR close?

Closes #.

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@korowa
Copy link
Owner Author

korowa commented Mar 26, 2024

/benchmark

@github-actions
Copy link

Benchmark results

Benchmarks comparing bd9b33c (main) and 29f1cca (PR)
Comparing bd9b33c and 29f1cca
--------------------
Benchmark tpch.json
--------------------
┏━━━━━━━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━━━━━━━┓
┃ Query        ┃  bd9b33c ┃  29f1cca ┃       Change ┃
┡━━━━━━━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━━━━━━━┩
│ QQuery 1     │ 441.95ms │ 444.16ms │    no change │
│ QQuery 2     │  58.42ms │  58.28ms │    no change │
│ QQuery 3     │ 144.94ms │ 146.66ms │    no change │
│ QQuery 4     │  88.37ms │  89.25ms │    no change │
│ QQuery 5     │ 201.16ms │ 202.38ms │    no change │
│ QQuery 6     │ 105.91ms │ 108.28ms │    no change │
│ QQuery 7     │ 286.99ms │ 286.74ms │    no change │
│ QQuery 8     │ 195.37ms │ 264.79ms │ 1.36x slower │
│ QQuery 9     │ 304.18ms │ 300.52ms │    no change │
│ QQuery 10    │ 233.48ms │ 237.65ms │    no change │
│ QQuery 11    │  63.45ms │  62.96ms │    no change │
│ QQuery 12    │ 127.10ms │ 128.03ms │    no change │
│ QQuery 13    │ 175.81ms │ 180.39ms │    no change │
│ QQuery 14    │ 128.84ms │ 131.94ms │    no change │
│ QQuery 15    │ 190.02ms │ 191.64ms │    no change │
│ QQuery 16    │  50.17ms │  51.52ms │    no change │
│ QQuery 17    │ 293.26ms │ 315.91ms │ 1.08x slower │
│ QQuery 18    │ 449.22ms │ 464.02ms │    no change │
│ QQuery 19    │ 236.13ms │ 235.18ms │    no change │
│ QQuery 20    │ 197.30ms │ 191.10ms │    no change │
│ QQuery 21    │ 326.52ms │ 320.96ms │    no change │
│ QQuery 22    │  53.66ms │  56.80ms │ 1.06x slower │
└──────────────┴──────────┴──────────┴──────────────┘
┏━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━┓
┃ Benchmark Summary      ┃           ┃
┡━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━┩
│ Total Time (bd9b33c)   │ 4352.26ms │
│ Total Time (29f1cca)   │ 4469.16ms │
│ Average Time (bd9b33c) │  197.83ms │
│ Average Time (29f1cca) │  203.14ms │
│ Queries Faster         │         0 │
│ Queries Slower         │         3 │
│ Queries with No Change │        19 │
└────────────────────────┴───────────┘

@korowa
Copy link
Owner Author

korowa commented Mar 26, 2024

/benchmark

@github-actions
Copy link

Benchmark results

Benchmarks comparing bd9b33c (main) and 29f1cca (PR)
Comparing bd9b33c and 29f1cca
--------------------
Benchmark tpch.json
--------------------
┏━━━━━━━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━━━━━━━━┓
┃ Query        ┃  bd9b33c ┃  29f1cca ┃        Change ┃
┡━━━━━━━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━━━━━━━━┩
│ QQuery 1     │ 442.26ms │ 448.79ms │     no change │
│ QQuery 2     │  58.09ms │  67.93ms │  1.17x slower │
│ QQuery 3     │ 145.22ms │ 203.54ms │  1.40x slower │
│ QQuery 4     │  89.05ms │ 117.94ms │  1.32x slower │
│ QQuery 5     │ 205.15ms │ 204.39ms │     no change │
│ QQuery 6     │ 107.24ms │ 107.59ms │     no change │
│ QQuery 7     │ 284.17ms │ 282.61ms │     no change │
│ QQuery 8     │ 193.53ms │ 196.78ms │     no change │
│ QQuery 9     │ 303.07ms │ 304.18ms │     no change │
│ QQuery 10    │ 238.86ms │ 239.06ms │     no change │
│ QQuery 11    │  63.41ms │  63.89ms │     no change │
│ QQuery 12    │ 128.75ms │ 126.71ms │     no change │
│ QQuery 13    │ 184.60ms │ 179.30ms │     no change │
│ QQuery 14    │ 129.76ms │ 131.91ms │     no change │
│ QQuery 15    │ 198.03ms │ 199.26ms │     no change │
│ QQuery 16    │  51.41ms │  51.82ms │     no change │
│ QQuery 17    │ 308.06ms │ 316.15ms │     no change │
│ QQuery 18    │ 452.86ms │ 456.74ms │     no change │
│ QQuery 19    │ 232.80ms │ 235.53ms │     no change │
│ QQuery 20    │ 191.79ms │ 195.64ms │     no change │
│ QQuery 21    │ 329.10ms │ 327.27ms │     no change │
│ QQuery 22    │  75.42ms │  53.58ms │ +1.41x faster │
└──────────────┴──────────┴──────────┴───────────────┘
┏━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━┓
┃ Benchmark Summary      ┃           ┃
┡━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━┩
│ Total Time (bd9b33c)   │ 4412.65ms │
│ Total Time (29f1cca)   │ 4510.59ms │
│ Average Time (bd9b33c) │  200.57ms │
│ Average Time (29f1cca) │  205.03ms │
│ Queries Faster         │         1 │
│ Queries Slower         │         3 │
│ Queries with No Change │        18 │
└────────────────────────┴───────────┘

@korowa korowa force-pushed the semijoin_statistics branch from 8cdb65e to 9bd30ea Compare March 28, 2024 19:07
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