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(ingestion/clickhouse-usage): add sql aggregator to clickhouse usage #12526

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Update clickhouse_usage.py

950110d
Select commit
Loading
Failed to load commit list.
Draft

feat(ingestion/clickhouse-usage): add sql aggregator to clickhouse usage #12526

Update clickhouse_usage.py
950110d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jan 31, 2025 in 0s

0.00% of diff hit (target 90.00%)

View this Pull Request on Codecov

0.00% of diff hit (target 90.00%)

Annotations

Check warning on line 24 in metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py#L24

Added line #L24 was not covered by tests

Check warning on line 28 in metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py#L27-L28

Added lines #L27 - L28 were not covered by tests

Check warning on line 33 in metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py#L33

Added line #L33 was not covered by tests

Check warning on line 90 in metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py#L90

Added line #L90 was not covered by tests

Check warning on line 110 in metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py#L110

Added line #L110 was not covered by tests

Check warning on line 125 in metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py#L124-L125

Added lines #L124 - L125 were not covered by tests

Check warning on line 143 in metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/usage/clickhouse_usage.py#L140-L143

Added lines #L140 - L143 were not covered by tests