Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

IGNITE-23969 Sql. Improve predicate selectivity estimation #5211

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

IGNITE-23969 Sql. Improve predicate selectivity estimation #5211

wants to merge 20 commits into from

Conversation

zstan
Copy link
Contributor

@zstan zstan commented Feb 11, 2025 •
edited
Loading

TpcdsBenchmark
--main:

Benchmark           (fsync)  (queryId)  Mode  Cnt     Score     Error  Units
TpcdsBenchmark.run    false          1  avgt   10   121.749 ±   7.461  ms/op
TpcdsBenchmark.run    false          2  avgt   10  3836.207 ± 207.361  ms/op
TpcdsBenchmark.run    false          3  avgt   10    48.116 ±   2.150  ms/op
TpcdsBenchmark.run    false          4  avgt   10  1612.781 ±  76.630  ms/op
TpcdsBenchmark.run    false         48  avgt   10  4551.836 ± 225.324  ms/op

--this pr:

Benchmark           (fsync)  (queryId)  Mode  Cnt     Score     Error  Units
TpcdsBenchmark.run    false          1  avgt   10   108.429 ±  16.108  ms/op
TpcdsBenchmark.run    false          2  avgt   10  3751.181 ± 350.560  ms/op
TpcdsBenchmark.run    false          3  avgt   10    47.588 ±   1.918  ms/op
TpcdsBenchmark.run    false          4  avgt   10  1517.953 ±  45.865  ms/op
TpcdsBenchmark.run    false         48  avgt   10    79.810 ±  2.701   ms/op

@zstan zstan marked this pull request as ready for review February 12, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ygerzhedovich ygerzhedovich ygerzhedovich left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zstan @ygerzhedovich