dy102 commented on code in PR #9234:
URL: https://github.com/apache/seatunnel/pull/9234#discussion_r2061953038
##########
seatunnel-connectors-v2/connector-maxcompute/src/main/java/org/apache/seatunnel/connectors/seatunnel/maxcompute/util/MaxcomputeTypeMapper.java:
##########
@@ -272,6 +272,9 @@ private static Object resolveObject2Maxcompute(Object
field, TypeInfo typeInfo)
case CHAR:
return new Char((String) field);
case STRING:
Review Comment:
Thank you for the suggestion!
Based on my understanding, the Timestamp value is already converted to
LocalDateTime during the SeaTunnelRow mapping process.
Therefore, the subsequent formatting operation only affects the string
output, without impacting the internal precision.
In this context, I believe an additional conversion here might be redundant.
🙏
--
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]
