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.

Sign up for GitHub

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] read_sql #1943

Merged
merged 30 commits into from
Mar 13, 2024

refactor and add limit pushdown

bd65b05
Select commit
Loading
Failed to load commit list.
Merged

[FEAT] read_sql #1943

refactor and add limit pushdown
bd65b05
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Mar 13, 2024 in 1s

31.73% of diff hit (target 84.67%)

View this Pull Request on Codecov

31.73% of diff hit (target 84.67%)

Annotations

Check warning on line 485 in daft/datatype.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/datatype.py#L485 

Added line #L485 was not covered by tests

Check warning on line 40 in daft/io/_sql.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/io/_sql.py#L39-L40 

Added lines #L39 - L40 were not covered by tests

Check warning on line 43 in daft/io/_sql.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/io/_sql.py#L42-L43 

Added lines #L42 - L43 were not covered by tests

Check warning on line 45 in daft/io/_sql.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/io/_sql.py#L45 

Added line #L45 was not covered by tests

Check warning on line 53 in daft/io/_sql.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/io/_sql.py#L52-L53 

Added lines #L52 - L53 were not covered by tests

Check warning on line 55 in daft/io/_sql.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/io/_sql.py#L55 

Added line #L55 was not covered by tests

Check warning on line 120 in daft/logical/schema.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/logical/schema.py#L120 

Added line #L120 was not covered by tests

Check warning on line 26 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L22-L26 

Added lines #L22 - L26 were not covered by tests

Check warning on line 35 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L29-L35 

Added lines #L29 - L35 were not covered by tests

Check warning on line 40 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L38-L40 

Added lines #L38 - L40 were not covered by tests

Check warning on line 42 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L42 

Added line #L42 was not covered by tests

Check warning on line 44 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L44 

Added line #L44 was not covered by tests

Check warning on line 47 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L46-L47 

Added lines #L46 - L47 were not covered by tests

Check warning on line 50 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L49-L50 

Added lines #L49 - L50 were not covered by tests

Check warning on line 52 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L52 

Added line #L52 was not covered by tests

Check warning on line 56 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L56 

Added line #L56 was not covered by tests

Check warning on line 69 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L68-L69 

Added lines #L68 - L69 were not covered by tests

Check warning on line 73 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L72-L73 

Added lines #L72 - L73 were not covered by tests

Check warning on line 75 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L75 

Added line #L75 was not covered by tests

Check warning on line 78 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L78 

Added line #L78 was not covered by tests

Check warning on line 85 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L80-L85 

Added lines #L80 - L85 were not covered by tests

Check warning on line 88 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L88 

Added line #L88 was not covered by tests

Check warning on line 95 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L90-L95 

Added lines #L90 - L95 were not covered by tests

Check warning on line 100 in daft/sql/sql_reader.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_reader.py#L98-L100 

Added lines #L98 - L100 were not covered by tests

Check warning on line 46 in daft/sql/sql_scan.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/sql/sql_scan.py#L40-L46 

Added lines #L40 - L46 were not covered by tests
View more details on Codecov