umeshdangat commented on code in PR #3434:
URL: https://github.com/apache/flink-cdc/pull/3434#discussion_r1861307255


##########
flink-cdc-runtime/src/main/java/org/apache/flink/cdc/runtime/serializer/schema/DataFieldSerializer.java:
##########
@@ -39,7 +39,14 @@ public class DataFieldSerializer extends 
TypeSerializerSingleton<DataField> {
     public static final DataFieldSerializer INSTANCE = new 
DataFieldSerializer();
 
     private final StringSerializer stringSerializer = 
StringSerializer.INSTANCE;
-    private final DataTypeSerializer dataTypeSerializer = new 
DataTypeSerializer();
+    private DataTypeSerializer dataTypeSerializer;
+
+    private DataTypeSerializer getDataTypeSerializer() {

Review Comment:
   > I came across a bug due to circular dependency between 
DataFieldSerializer, DataTypeSerializer, and RowTypeSerializer classes that 
causes NPE for certain RowTypes like nested row type. I have baked that change 
in this PR but also have a separate one 
https://github.com/apache/flink-cdc/pull/3400
   
   [explained earlier 
here](https://github.com/apache/flink-cdc/pull/3434#issuecomment-2192072774)
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to