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

Jump to bottom

[Source-postgres] : Source operations suport for meta column #36432

Merged
merged 28 commits into from
Mar 26, 2024
Merged

[Source-postgres] : Source operations suport for meta column #36432

merged 28 commits into from
Mar 26, 2024

Conversation

akashkulk
Copy link
Contributor

@akashkulk akashkulk commented Mar 24, 2024 •
edited
Loading

One bad value should not fail the sync.
Instead, null out the offending column and quarantine the value into a airbyte_changes.meta column

@vercel Vercel
Copy link

vercel bot commented Mar 24, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Mar 26, 2024 5:00pm

@octavia-squidington-iii octavia-squidington-iii added area/connectors Connector related issues CDK Connector Development Kit connectors/source/mysql labels Mar 24, 2024
@octavia-squidington-iii octavia-squidington-iii removed the area/connectors Connector related issues label Mar 24, 2024
@octavia-squidington-iii octavia-squidington-iii added the area/connectors Connector related issues label Mar 24, 2024
@akashkulk akashkulk changed the title [Source-postgres] : Source operations suport for met [Source-postgres] : Source operations suport for meta column Mar 25, 2024
@akashkulk akashkulk requested a review from rodireich March 25, 2024 17:24
@akashkulk akashkulk marked this pull request as ready for review March 25, 2024 18:54
@akashkulk akashkulk requested review from a team as code owners March 25, 2024 18:54
@akashkulk
Copy link
Contributor Author

akashkulk commented Mar 26, 2024 •
edited by github-actions bot
Loading

/publish-java-cdk

🕑 https://github.com/airbytehq/airbyte/actions/runs/8439130190
❌ Publish Java CDK version=0.27.5 failed!

@akashkulk
Copy link
Contributor Author

akashkulk commented Mar 26, 2024 •
edited by github-actions bot
Loading

/publish-java-cdk

🕑 https://github.com/airbytehq/airbyte/actions/runs/8439374585
✅ Successfully published Java CDK version=0.27.6!

@akashkulk akashkulk merged commit 9dee837 into master Mar 26, 2024
29 checks passed
@akashkulk akashkulk deleted the akash/pgrefac-source-ops branch March 26, 2024 17:27
stephane-airbyte added a commit that referenced this pull request Mar 27, 2024
@stephane-airbyte
migrate #36432 to kotlin (1/2)
f61e692
stephane-airbyte added a commit that referenced this pull request Mar 27, 2024
@stephane-airbyte
convert #36432 to kotlin 2/2
0972a11
stephane-airbyte added a commit that referenced this pull request Mar 27, 2024
@stephane-airbyte
migrate #36432 to kotlin (1/2)
2509237
stephane-airbyte added a commit that referenced this pull request Mar 27, 2024
@stephane-airbyte
convert #36432 to kotlin 2/2
6372cf6
stephane-airbyte added a commit that referenced this pull request Mar 27, 2024
@stephane-airbyte
convert #36432 to kotlin 2/2
fa42a85
stephane-airbyte added a commit that referenced this pull request Mar 27, 2024
@stephane-airbyte
convert #36432 to kotlin 2/2
4689992
stephane-airbyte added a commit that referenced this pull request Mar 27, 2024
@stephane-airbyte
convert #36432 to kotlin 2/2
3f8fe7d
stephane-airbyte added a commit that referenced this pull request Mar 27, 2024
@stephane-airbyte
convert #36432 to kotlin 2/2
b618796
stephane-airbyte added a commit that referenced this pull request Mar 27, 2024
@stephane-airbyte
convert #36432 to kotlin 2/2
c3100fe
nurikk pushed a commit to nurikk/airbyte that referenced this pull request Apr 4, 2024
@akashkulk @nurikk-sa
[Source-postgres] : Source operations suport for meta column (airbyte…
5537b90 
…hq#36432)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rodireich rodireich rodireich approved these changes

Assignees
No one assigned
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation CDK Connector Development Kit connectors/source/mysql connectors/source/postgres
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@akashkulk @rodireich @octavia-squidington-iii