Skip to content

Conversation

@kevinkarchacryl
Copy link
Contributor

Should surface this as a warning in the UI instead of an invisible error in the ingestion logs.

@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Nov 6, 2025
@codecov
Copy link

codecov bot commented Nov 6, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ngestion/src/datahub/ingestion/source/s3/source.py 0.00% 1 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 the needs-review Label for PRs that need review from a maintainer. label Nov 6, 2025
@datahub-cyborg datahub-cyborg bot added pending-submitter-merge and removed needs-review Label for PRs that need review from a maintainer. labels Nov 12, 2025
Co-authored-by: Sergio Gómez Villamor <sgomezvillamor@gmail.com>
@kevinkarchacryl kevinkarchacryl merged commit 75ee3c9 into master Nov 13, 2025
61 of 62 checks passed
@kevinkarchacryl kevinkarchacryl deleted the warn-s3-no-schema-extract branch November 13, 2025 14:42
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