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.

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

[BUG] Fix runner check at plan execution time for new query planner #1435

Merged
merged 1 commit into from
Sep 25, 2023

Propagate runner flag from client.

8d6067b
Select commit
Loading
Failed to load commit list.
Merged

[BUG] Fix runner check at plan execution time for new query planner #1435

Propagate runner flag from client.
8d6067b
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
Codecov / codecov/project succeeded Sep 25, 2023 in 0s

74.51% (-0.01%) compared to ecb596a

View this Pull Request on Codecov

74.51% (-0.01%) compared to ecb596a

Details

Codecov Report

Merging #1435 (8d6067b) into main (ecb596a) will decrease coverage by 0.01%.
The diff coverage is 85.71%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1435      +/-   ##
==========================================
- Coverage   74.51%   74.51%   -0.01%     
==========================================
  Files          60       60              
  Lines        6035     6034       -1     
==========================================
- Hits         4497     4496       -1     
  Misses       1538     1538              
Files Coverage Δ
daft/execution/rust_physical_plan_shim.py 98.18% <100.00%> (-0.04%) ⬇️
daft/planner/planner.py 87.50% <100.00%> (ø)
daft/planner/rust_planner.py 100.00% <100.00%> (ø)
daft/runners/pyrunner.py 95.80% <100.00%> (ø)
daft/runners/ray_runner.py 91.20% <100.00%> (ø)
daft/planner/py_planner.py 0.00% <0.00%> (ø)