Uh oh!
There was an error while loading. Please reload this page.
[Improve](streaming job) support postgres partition table sync - #60560
Conversation
Thearas
commented
Feb 6, 2026
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
JNSimba
commented
Feb 6, 2026
run buildall |
There was a problem hiding this comment.
Pull request overview
This PR enhances PostgreSQL partition table synchronization in streaming jobs by adding support for publish_via_partition_root parameter when creating publication instances for PostgreSQL 13+. This is a follow-up to PR #59461 which added PostgreSQL synchronization support.
Changes:
- Added support for PostgreSQL partition tables by enabling
publish_via_partition_rootin publications for PostgreSQL 13+ - Enhanced database configuration to support flexible database specification (either in JDBC URL or as a separate parameter)
- Added comprehensive regression test for partition table synchronization including snapshot, incremental changes, and dynamic partition creation
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| test_streaming_postgres_job_partition.groovy | New regression test validating partition table synchronization with insert, update, delete operations and dynamic partition creation |
| test_streaming_postgres_job_partition.out | Expected output for the partition table test showing snapshot and binlog data |
| PostgresSourceReader.java | Enhanced database configuration flexibility and enabled partition table scanning via setIncludePartitionedTables(true) |
| PostgresReplicationConnection.java | Copied and modified Debezium file adding publish_via_partition_root parameter for PostgreSQL 13+ in ALL_TABLES publication mode |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
JNSimba
commented
Feb 6, 2026
run buildall |
doris-robot
commented
Feb 6, 2026
TPC-H: Total hot run time: 30509 ms |
doris-robot
commented
Feb 6, 2026
ClickBench: Total hot run time: 28.56 s |
JNSimba
commented
Feb 6, 2026
run external |
JNSimba
commented
Feb 6, 2026
run vault_p0 |
JNSimba
commented
Feb 6, 2026
run nonConcurrent |
hello-stephen
commented
Feb 6, 2026
FE Regression Coverage ReportIncrement line coverage |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
Uh oh!
There was an error while loading. Please reload this page.
### What problem does this PR solve? Related PR: #59461 To enhance partition table synchronization, add `publish_via_partition_root` when creating a PUBLICATION instance, specifically for PG 13+.
…e#60560) ### What problem does this PR solve? Related PR: apache#59461 To enhance partition table synchronization, add `publish_via_partition_root` when creating a PUBLICATION instance, specifically for PG 13+.
… sync #60560 (#60614) ### What problem does this PR solve? Cherry-picked from #60560, #60624 ### Release note None ### Check List (For Author) - Test <!-- At least one of them must be included. --> - [ ] Regression test - [ ] Unit Test - [ ] Manual test (add detailed scripts or steps below) - [ ] No need to test or manual test. Explain why: - [ ] This is a refactor/code format and no logic has been changed. - [ ] Previous test can cover this change. - [ ] No code files have been changed. - [ ] Other reason <!-- Add your reason? --> - Behavior changed: - [ ] No. - [ ] Yes. <!-- Explain the behavior change --> - Does this need documentation? - [ ] No. - [ ] Yes. <!-- Add document PR link here. eg: apache/doris-website#1214 --> ### Check List (For Reviewer who merge this PR) - [ ] Confirm the release note - [ ] Confirm test cases - [ ] Confirm document - [ ] Add branch pick label <!-- Add branch pick label that this PR should merge into -->
… sync apache#60560 (apache#60614) ### What problem does this PR solve? Cherry-picked from apache#60560, apache#60624 ### Release note None ### Check List (For Author) - Test <!-- At least one of them must be included. --> - [ ] Regression test - [ ] Unit Test - [ ] Manual test (add detailed scripts or steps below) - [ ] No need to test or manual test. Explain why: - [ ] This is a refactor/code format and no logic has been changed. - [ ] Previous test can cover this change. - [ ] No code files have been changed. - [ ] Other reason <!-- Add your reason? --> - Behavior changed: - [ ] No. - [ ] Yes. <!-- Explain the behavior change --> - Does this need documentation? - [ ] No. - [ ] Yes. <!-- Add document PR link here. eg: apache/doris-website#1214 --> ### Check List (For Reviewer who merge this PR) - [ ] Confirm the release note - [ ] Confirm test cases - [ ] Confirm document - [ ] Add branch pick label <!-- Add branch pick label that this PR should merge into -->
What problem does this PR solve?
Related PR: #59461
To enhance partition table synchronization, add
publish_via_partition_rootwhen creating a PUBLICATION instance, specifically for PG 13+.Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)