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/mode): Handle 204 response and invalid json #12156

Merged
merged 9 commits into from
Dec 24, 2024
+151 −36

fix test

c05718f
Select commit
Loading
Failed to load commit list.
Merged

fix(ingest/mode): Handle 204 response and invalid json #12156

fix test
c05718f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Dec 24, 2024 in 1s

35.71% of diff hit (target 59.33%)

View this Pull Request on Codecov

35.71% of diff hit (target 59.33%)

Annotations

Check warning on line 335 in metadata-ingestion/src/datahub/ingestion/source/mode.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/mode.py#L335 

Added line #L335 was not covered by tests

Check warning on line 578 in metadata-ingestion/src/datahub/ingestion/source/mode.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/mode.py#L578 

Added line #L578 was not covered by tests

Check warning on line 728 in metadata-ingestion/src/datahub/ingestion/source/mode.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/mode.py#L728 

Added line #L728 was not covered by tests

Check warning on line 819 in metadata-ingestion/src/datahub/ingestion/source/mode.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/mode.py#L819 

Added line #L819 was not covered by tests

Check warning on line 1389 in metadata-ingestion/src/datahub/ingestion/source/mode.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/mode.py#L1389 

Added line #L1389 was not covered by tests

Check warning on line 1407 in metadata-ingestion/src/datahub/ingestion/source/mode.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/mode.py#L1407 

Added line #L1407 was not covered by tests

Check warning on line 1423 in metadata-ingestion/src/datahub/ingestion/source/mode.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/mode.py#L1423 

Added line #L1423 was not covered by tests

Check warning on line 1440 in metadata-ingestion/src/datahub/ingestion/source/mode.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/mode.py#L1440 

Added line #L1440 was not covered by tests

Check warning on line 1458 in metadata-ingestion/src/datahub/ingestion/source/mode.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

metadata-ingestion/src/datahub/ingestion/source/mode.py#L1458 

Added line #L1458 was not covered by tests
View more details on Codecov