Skip to content

Conversation

fuyun2024
Copy link
Contributor

fix this pr : #2178

@GOODBOY008 GOODBOY008 self-requested a review June 13, 2023 10:05
Copy link
Member

@GOODBOY008 GOODBOY008 left a comment

Choose a reason for hiding this comment

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

@fuyun2024 I have left some comments.

public class RecordsFormatter {

private final DataType dataType;
private final ZoneId zoneId;
Copy link
Member

Choose a reason for hiding this comment

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

Remove useless field.

private TypeInformation<RowData> typeInfo;
private DebeziumDeserializationSchema<RowData> deserializationSchema;
private SimpleCollector collector;
private RowRowConverter rowRowConverter;
Copy link
Member

Choose a reason for hiding this comment

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

Add final to this fields.

Copy link
Member

@GOODBOY008 GOODBOY008 left a comment

Choose a reason for hiding this comment

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

LGTM

@GOODBOY008 GOODBOY008 merged commit 66556ee into apache:master Jun 16, 2023
ChaomingZhangCN pushed a commit to ChaomingZhangCN/flink-cdc that referenced this pull request Jan 13, 2025
… reading process (apache#2179)

* [sqlserver] Merge high and low watermark data during snapshot reading process
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants