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

[CHORE] Better logging for physical plan #1499

Merged
merged 2 commits into from
Oct 17, 2023

Fix PartitionTaskBuilder as well

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

[CHORE] Better logging for physical plan #1499

Fix PartitionTaskBuilder as well
0a48456
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Oct 17, 2023 in 0s

7.69% of diff hit (target 74.87%)

View this Pull Request on Codecov

7.69% of diff hit (target 74.87%)

Annotations

Check warning on line 1 in daft/logging.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/logging.py#L1

Added line #L1 was not covered by tests

Check warning on line 3 in daft/logging.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/logging.py#L3

Added line #L3 was not covered by tests

Check warning on line 6 in daft/logging.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/logging.py#L6

Added line #L6 was not covered by tests

Check warning on line 11 in daft/logging.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/logging.py#L10-L11

Added lines #L10 - L11 were not covered by tests

Check warning on line 15 in daft/logging.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/logging.py#L13-L15

Added lines #L13 - L15 were not covered by tests

Check warning on line 20 in daft/logging.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/logging.py#L17-L20

Added lines #L17 - L20 were not covered by tests