Eventual-Inc / Daft

Distributed data engine for Python/SQL designed for the cloud, powered by Rust
https://getdaft.io
Apache License 2.0
2.34k stars 164 forks source link

[CHORE] Add TPC-H queries in SQL #3392

Closed kevinzwang closed 14 hours ago

codspeed-hq[bot] commented 14 hours ago

CodSpeed Performance Report

Merging #3392 will degrade performances by 34.84%

Comparing kevin/tpch-sql-queries (a99cb1b) with main (e236332)

Summary

⚡ 1 improvements
❌ 1 regressions
✅ 15 untouched benchmarks

:warning: Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark main kevin/tpch-sql-queries Change
test_iter_rows_first_row[100 Small Files] 330.7 ms 246.1 ms +34.38%
test_show[100 Small Files] 15.5 ms 23.8 ms -34.84%
codecov[bot] commented 14 hours ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.35%. Comparing base (e236332) to head (a99cb1b). Report is 1 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/3392/graphs/tree.svg?width=650&height=150&src=pr&token=J430QVFE89&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc)](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/3392?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc) ```diff @@ Coverage Diff @@ ## main #3392 +/- ## ========================================== - Coverage 77.36% 77.35% -0.01% ========================================== Files 685 685 Lines 83639 83637 -2 ========================================== - Hits 64704 64698 -6 - Misses 18935 18939 +4 ``` [see 4 files with indirect coverage changes](https://app.codecov.io/gh/Eventual-Inc/Daft/pull/3392/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Eventual-Inc)

🚨 Try these New Features:

graphite-app[bot] commented 14 hours ago

Graphite Automations

"Request reviewers once CI passes" took an action on this PR • (11/21/24)

1 reviewer was added to this PR based on Andrew Gazelka's automation.