Skip to content

Conversation

treff7es
Copy link
Contributor

@treff7es treff7es commented May 7, 2025

Only report a warning if the queries endpoint of a report returns 404.
This can happen if a report gets deleted.

@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label May 7, 2025
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label May 7, 2025
Copy link

codecov bot commented May 7, 2025

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ata-ingestion/src/datahub/ingestion/source/mode.py 0.00% 3 Missing ⚠️

❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage.

📢 Thoughts on this report? Let us know!

@datahub-cyborg datahub-cyborg bot added pending-submitter-merge and removed needs-review Label for PRs that need review from a maintainer. labels May 7, 2025
@treff7es treff7es merged commit f7ea0b9 into master May 7, 2025
61 of 63 checks passed
@treff7es treff7es deleted the mode_missing_query_warning branch May 7, 2025 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ingestion PR or Issue related to the ingestion of metadata pending-submitter-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants