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

[FEAT] parquet reader refactor, add parquet_stats_reader and parquet_schema_reader (1/2) #1191

Merged
merged 13 commits into from
Jul 27, 2023

dont unwrap error

0e5a9c2
Select commit
Loading
Failed to load commit list.
Merged

[FEAT] parquet reader refactor, add parquet_stats_reader and parquet_schema_reader (1/2) #1191

dont unwrap error
0e5a9c2
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 Jul 26, 2023 in 1s

88.37% (-0.07%) compared to b13bdf8

View this Pull Request on Codecov

88.37% (-0.07%) compared to b13bdf8

Details

Codecov Report

Merging #1191 (0e5a9c2) into main (b13bdf8) will decrease coverage by 0.07%.
Report is 2 commits behind head on main.
The diff coverage is 61.53%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1191      +/-   ##
==========================================
- Coverage   88.44%   88.37%   -0.07%     
==========================================
  Files          54       54              
  Lines        5564     5576      +12     
==========================================
+ Hits         4921     4928       +7     
- Misses        643      648       +5     
Files Changed Coverage Δ
daft/table/table.py 87.50% <50.00%> (-0.97%) ⬇️
daft/logical/schema.py 92.30% <71.42%> (-1.81%) ⬇️