Sqllogictest: use the same normalization for all tests#5013
Sqllogictest: use the same normalization for all tests#5013alamb merged 2 commits intoapache:masterfrom
Conversation
alamb
left a comment
There was a problem hiding this comment.
Looks great to me -- thank you @melgenek
cc @xudong963
| query R | ||
| SELECT avg(c12) FROM aggregate_test_100 | ||
| ---- | ||
| 0.5089725099127211 |
|
Thanks again @melgenek -- these improvements are really nice |
|
Benchmark runs are scheduled for baseline = 407caad and contender = 8dc4c96. 8dc4c96 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
Which issue does this PR close?
Closes #5010.
Rationale for this change
This change simplifies the sqllogictest results normalization logic. It is now the same for all test files.
What changes are included in this PR?
Are these changes tested?
There are Github Actions for both Datafusion and Postgres sqllogictest runners.
Are there any user-facing changes?
No.