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/gc): misc fixes in gc source #12226

Merged
merged 2 commits into from
Dec 26, 2024
+68 −17

minor change

ebd8e08
Select commit
Loading
Failed to load commit list.
Merged

fix(ingest/gc): misc fixes in gc source #12226

minor change
ebd8e08
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Dec 26, 2024 in 1s

2.17% of diff hit (target 74.95%)

View this Pull Request on Codecov

2.17% of diff hit (target 74.95%)

Annotations

Check warning on line 37 in metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py#L37 

Added line #L37 was not covered by tests

Check warning on line 144 in metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py#L144 

Added line #L144 was not covered by tests

Check warning on line 150 in metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py#L150 

Added line #L150 was not covered by tests

Check warning on line 156 in metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py#L156 

Added line #L156 was not covered by tests

Check warning on line 166 in metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py#L166 

Added line #L166 was not covered by tests

Check warning on line 175 in metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py#L170-L175 

Added lines #L170 - L175 were not covered by tests

Check warning on line 177 in metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py#L177 

Added line #L177 was not covered by tests

Check warning on line 296 in metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/datahub_gc.py#L295-L296 

Added lines #L295 - L296 were not covered by tests

Check warning on line 1 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L1 

Added line #L1 was not covered by tests

Check warning on line 46 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L46 

Added line #L46 was not covered by tests

Check warning on line 51 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L51 

Added line #L51 was not covered by tests

Check warning on line 67 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L61-L67 

Added lines #L61 - L67 were not covered by tests

Check warning on line 143 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L140-L143 

Added lines #L140 - L143 were not covered by tests

Check warning on line 146 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L146 

Added line #L146 was not covered by tests

Check warning on line 164 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L164 

Added line #L164 was not covered by tests

Check warning on line 173 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L173 

Added line #L173 was not covered by tests

Check warning on line 194 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L194 

Added line #L194 was not covered by tests

Check warning on line 202 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L202 

Added line #L202 was not covered by tests

Check warning on line 211 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L211 

Added line #L211 was not covered by tests

Check warning on line 223 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L223 

Added line #L223 was not covered by tests

Check warning on line 233 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L233 

Added line #L233 was not covered by tests

Check warning on line 239 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L238-L239 

Added lines #L238 - L239 were not covered by tests

Check warning on line 249 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L247-L249 

Added lines #L247 - L249 were not covered by tests

Check warning on line 257 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L257 

Added line #L257 was not covered by tests

Check warning on line 270 in metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/gc/execution_request_cleanup.py#L269-L270 

Added lines #L269 - L270 were not covered by tests
View more details on Codecov