-
Notifications
You must be signed in to change notification settings - Fork 67
fix: resolve issue where some destinations will fail due to null/missing stream namespace
keys in catalog
#748
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.
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
Draft
aaronsteers
wants to merge
25
commits into
main
Choose a base branch
from
devin/1755114459-fix-pyairbyte-null-namespace
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 9 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
4e24489
spike: Replace null namespaces with empty strings
devin-ai-integration[bot] 054093a
fix: Shorten comment to resolve Ruff lint error (line length)
devin-ai-integration[bot] 0c2f196
Update airbyte/_message_iterators.py
aaronsteers 5c05e1d
Update airbyte/_message_iterators.py
aaronsteers 761554e
fix: Replace null namespaces with empty strings in catalog validation
devin-ai-integration[bot] 3a2cb45
fix: Also handle null source_defined_cursor and is_resumable fields
devin-ai-integration[bot] d098050
fix: Clean up null namespaces in record messages from external sources
devin-ai-integration[bot] 201187a
fix: Clean up null namespaces in record messages from external source…
devin-ai-integration[bot] 808ec52
fix: Clean up null namespaces in record messages at source execution …
devin-ai-integration[bot] 9aad1fb
fix: Comment out performance-problematic namespace checks in high-vol…
devin-ai-integration[bot] 85a3b4e
fix: Ensure catalog cleaning changes are included
devin-ai-integration[bot] 4c5a6d2
fix: Use Pydantic exclude_none=True to omit null fields from catalog …
devin-ai-integration[bot] 50b07ad
fix: Add exclude_none=True to remaining model_dump_json() calls
devin-ai-integration[bot] 89ff68a
fix: Complete exclude_none=True implementation for all model_dump_jso…
devin-ai-integration[bot] 9c4f594
fix: Clean up null namespaces in trace stream_status messages
devin-ai-integration[bot] 82e7730
Apply suggestion from @aaronsteers
aaronsteers 405b947
Apply suggestion from @aaronsteers
aaronsteers 35d9d60
Apply suggestion from @aaronsteers
aaronsteers d0a6f24
Apply suggestion from @aaronsteers
aaronsteers a8205c4
Apply suggestion from @aaronsteers
aaronsteers c8d1d41
feat: Switch benchmark default from source-e2e-test to source-faker
devin-ai-integration[bot] c3dce14
fix: Simplify source-faker config to use only count and seed parameters
devin-ai-integration[bot] 88756af
feat: Test catalog-only exclude_none theory, update get_benchmark_sou…
devin-ai-integration[bot] 57c9863
fix: Address lint and type checking issues
devin-ai-integration[bot] dbb96d0
security: Replace insecure tempfile.mktemp() with secure tempfile.mks…
devin-ai-integration[bot] File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.