Skip to content

[FLINK-38520][postgres] Postgres YAML CDC support array with null element#4254

Merged
lvyanquan merged 1 commit into
apache:masterfrom
Hisoka-X:FLINK-38520-pg-null
Feb 2, 2026
Merged

[FLINK-38520][postgres] Postgres YAML CDC support array with null element#4254
lvyanquan merged 1 commit into
apache:masterfrom
Hisoka-X:FLINK-38520-pg-null

Conversation

@Hisoka-X
Copy link
Copy Markdown
Member

@Hisoka-X Hisoka-X commented Feb 1, 2026

This close https://issues.apache.org/jira/browse/FLINK-38520

This pull request improves the handling of PostgreSQL array types in Flink CDC, specifically ensuring that arrays containing null elements are correctly deserialized and tested. It also refines error handling for unsupported multidimensional arrays.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances PostgreSQL array handling in Flink CDC to properly support arrays containing null elements. The changes refactor error handling for multidimensional arrays and ensure that arrays with null values are correctly deserialized.

Changes:

  • Refactored array conversion logic to check for multidimensional arrays upfront and simplified null element handling
  • Fixed grammatical error in multidimensional array error message ("Unable convert" → "Unable to convert")
  • Added comprehensive test coverage for arrays with null elements including SQL DDL and integration test

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
DebeziumEventDeserializationSchema.java Refactored convertToArray method to handle null elements correctly and check for multidimensional arrays earlier
column_type_test.sql Added new test table array_types_with_null with sample data containing null elements in arrays
PostgresFullTypesITCase.java Added testArrayTypesWithNull test method and updated error message assertion in existing test

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@lvyanquan lvyanquan left a comment

Choose a reason for hiding this comment

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

LGTM.

@lvyanquan lvyanquan merged commit 93e5547 into apache:master Feb 2, 2026
21 of 22 checks passed
@Hisoka-X Hisoka-X deleted the FLINK-38520-pg-null branch February 2, 2026 02:40
ThorneANN pushed a commit to ThorneANN/flink-cdc that referenced this pull request Mar 19, 2026
Mrart pushed a commit to Mrart/flink-cdc that referenced this pull request Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants