search

apache / datafusion-comet

Apache DataFusion Comet Spark Accelerator
https://datafusion.apache.org/comet
Apache License 2.0
823 stars 163 forks source link

build: fix build #917

Closed andygrove closed 2 months ago

andygrove commented 2 months ago

Which issue does this PR close?

N/A

Rationale for this change

Build is failing in main branch

What changes are included in this PR?

How are these changes tested?

codecov-commenter commented 2 months ago

Codecov Report

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

Project coverage is 54.97%. Comparing base (033fe6f) to head (53ec0a7). Report is 9 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #917 +/- ## ============================================= + Coverage 34.03% 54.97% +20.94% + Complexity 883 852 -31 ============================================= Files 113 109 -4 Lines 43170 10681 -32489 Branches 9516 2051 -7465 ============================================= - Hits 14693 5872 -8821 + Misses 25471 3775 -21696 + Partials 3006 1034 -1972 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

viirya commented 2 months ago

Thanks @andygrove @comphead