Skip to content

Revert "Revert "[source-postgres] Fix duplicate streams in postgres"" - #41067

Merged
Xiaohan Song (xiaohansong) merged 3 commits into
masterfrom
revert-40757-revert-40719-xiaohan/statefix
Jul 9, 2024
Merged

Xiaohan Song (xiaohansong) merged 3 commits into
masterfrom
revert-40757-revert-40719-xiaohan/statefix

Conversation

@xiaohansong

@xiaohansong Xiaohan Song (xiaohansong) commented Jul 9, 2024 •

Copy link
Copy Markdown
Contributor

Reverts #40757

This time I addressed the NPE error.

@vercel

vercel Bot commented Jul 9, 2024 •

Copy link
Copy Markdown

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

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 9, 2024 7:24pm

}

private boolean isIncrementalStream(final AirbyteStreamNameNamespacePair pair) {
return !resumableFullRefreshStreams.contains(pair) && !nonResumableFullRefreshStreams.contains(pair);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this function!

Could we have one line of comment here to explain this?

@xiaohansong
Xiaohan Song (xiaohansong) merged commit 5f42d38 into master Jul 9, 2024
@xiaohansong
Xiaohan Song (xiaohansong) deleted the revert-40757-revert-40719-xiaohan/statefix branch July 9, 2024 21:09

This branch was successfully deployed

1 active deployment
Preview — f07a8e52 Deployed Jul 9, 2024 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/postgres

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants