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] Enable feature-flagged native downloader in daft.read_parquet #1190

Merged
merged 13 commits into from
Jul 31, 2023

Test native downloader for Parquet reads in test_creation.py

429b05c
Select commit
Loading
Failed to load commit list.
Merged

[FEAT] Enable feature-flagged native downloader in daft.read_parquet #1190

Test native downloader for Parquet reads in test_creation.py
429b05c
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/patch failed Jul 27, 2023 in 0s

78.57% of diff hit (target 88.37%)

View this Pull Request on Codecov

78.57% of diff hit (target 88.37%)

Annotations

Check warning on line 9 in daft/datasources.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/datasources.py#L9

Added line #L9 was not covered by tests

Check warning on line 14 in daft/io/parquet.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/io/parquet.py#L14

Added line #L14 was not covered by tests

Check warning on line 20 in daft/table/schema_inference.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/table/schema_inference.py#L20

Added line #L20 was not covered by tests

Check warning on line 24 in daft/table/table_io.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/table/table_io.py#L24

Added line #L24 was not covered by tests

Check warning on line 116 in daft/table/table_io.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/table/table_io.py#L115-L116

Added lines #L115 - L116 were not covered by tests