linjianchang commented on code in PR #3995:
URL: https://github.com/apache/flink-cdc/pull/3995#discussion_r2681551553
##########
flink-cdc-connect/flink-cdc-source-connectors/flink-connector-debezium/src/main/java/org/apache/flink/cdc/debezium/event/DebeziumEventDeserializationSchema.java:
##########
@@ -371,7 +372,7 @@ protected Object convertToLocalTimeZoneTimestamp(Object
dbzObj, Schema schema) {
if (dbzObj instanceof String) {
String str = (String) dbzObj;
// TIMESTAMP_LTZ type is encoded in string type
- Instant instant = Instant.parse(str);
+ Instant instant = ZonedTimestamp.FORMATTER.parse(str,
Instant::from);
Review Comment:
> Could you add some explanation about why this change is necessary?
The main issue to be addressed is that while the number type in MySQL has
negative numbers in Scala, the decimal type does not.
<img width="1699" height="1139" alt="企业微信截图_17679489006213"
src="https://github.com/user-attachments/assets/6fa99098-b04f-44cb-91b9-397106056652";
/>
All sources in the pipeline have been tested with fulltype's ut and are
normal. This change has not affected other connectors.
<img width="730" height="396" alt="企业微信截图_17679538297312"
src="https://github.com/user-attachments/assets/3f9c09ee-cb50-4662-b7fe-5a8e52014102";
/>
<img width="758" height="569" alt="企业微信截图_17679536267111"
src="https://github.com/user-attachments/assets/3c502a10-ddd4-480a-a91c-48b698014095";
/>
--
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]
