yuxiqian commented on PR #4530: URL: https://github.com/apache/flink-cdc/pull/4530#issuecomment-5806723634
After carefully reviewed this PR, I believe it's not possible to extend TIME precision without breaking backwards compatibility. As AsyncTransformOperator may serialize DataChangeEvents into states, it's not safe to change the underlying type from INT to LONG, as TIME(>3) fields could not be deserialized correctly, as current serializer isn't versioned. The original Flink PR https://github.com/apache/flink/pull/26954#discussion_r2316303536 didn't modify it, too. Thus, I suggest postponing this PR for now. cc @leonardBang @ruanhang1993 WDYT? -- 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]
