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

fix(ingest/unity): add row count in table profile of delta tables #12480

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
+44 −7

Update metadata-ingestion/src/datahub/ingestion/source/ge_profiling_c…

007ab22
Select commit
Loading
Failed to load commit list.
Open

fix(ingest/unity): add row count in table profile of delta tables #12480

Update metadata-ingestion/src/datahub/ingestion/source/ge_profiling_c…
007ab22
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jan 28, 2025 in 1s

7.14% of diff hit (target 90.00%)

View this Pull Request on Codecov

7.14% of diff hit (target 90.00%)

Annotations

Check warning on line 147 in metadata-ingestion/src/datahub/ingestion/source/unity/ge_profiler.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/unity/ge_profiler.py#L147 

Added line #L147 was not covered by tests

Check warning on line 156 in metadata-ingestion/src/datahub/ingestion/source/unity/ge_profiler.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/unity/ge_profiler.py#L153-L156 

Added lines #L153 - L156 were not covered by tests

Check warning on line 191 in metadata-ingestion/src/datahub/ingestion/source/unity/ge_profiler.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/unity/ge_profiler.py#L191 

Added line #L191 was not covered by tests

Check warning on line 198 in metadata-ingestion/src/datahub/ingestion/source/unity/ge_profiler.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/unity/ge_profiler.py#L196-L198 

Added lines #L196 - L198 were not covered by tests

Check warning on line 203 in metadata-ingestion/src/datahub/ingestion/source/unity/ge_profiler.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/unity/ge_profiler.py#L200-L203 

Added lines #L200 - L203 were not covered by tests
View more details on Codecov