close
Skip to content

[bugFix][Connector-V2][CDC] SeaTunnelRowDebeziumDeserializationConverters NPE#7119

Merged
Hisoka-X merged 2 commits into
apache:devfrom
Asura7969:dev
Jul 8, 2024
Merged

[bugFix][Connector-V2][CDC] SeaTunnelRowDebeziumDeserializationConverters NPE#7119
Hisoka-X merged 2 commits into
apache:devfrom
Asura7969:dev

Conversation

@Asura7969

Copy link
Copy Markdown
Contributor

Purpose of this pull request

close #6901
ref #7113

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

Hisoka-X
Hisoka-X previously approved these changes Jul 7, 2024
delete.setTableId(tableId);
collector.collect(delete);
} else {
} else if (operation == Envelope.Operation.UPDATE) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

In addition to update related statements, are there any other statements that can reach the else block?

@Asura7969 Asura7969 Jul 7, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

TRUNCATE or MESSAGE
Can we ignore these two operations? then like:

 else {
      log.warn("Received {} operation, skip", operation);
  }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, we should ignore it. cc @hailin0

@hailin0 hailin0 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Hisoka-X Hisoka-X merged commit ae81879 into apache:dev Jul 8, 2024
hawk9821 pushed a commit to hawk9821/seatunnel that referenced this pull request Jul 10, 2024
Thomas-HuWei pushed a commit to zilliztech/seatunnel that referenced this pull request Jul 10, 2024
Thomas-HuWei pushed a commit to zilliztech/seatunnel that referenced this pull request Jul 10, 2024
hawk9821 pushed a commit to hawk9821/seatunnel that referenced this pull request Jul 13, 2024
chaorongzhi pushed a commit to chaorongzhi/seatunnel that referenced this pull request Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

postgreCDC TO hive

3 participants