yuxiqian commented on code in PR #3819:
URL: https://github.com/apache/flink-cdc/pull/3819#discussion_r1900673509
##########
flink-cdc-common/src/main/java/org/apache/flink/cdc/common/utils/DateTimeUtils.java:
##########
@@ -109,7 +119,7 @@ private static long internalParseTimestampMillis(String
dateStr, String format,
} catch (ParseException e) {
LOG.error(
String.format(
- "Exception when parsing datetime string '%s' in
format '%s'",
+ "Exception when parsing datetime string '%s' in
format '%s', the default value Long.MIN_VALUE(-9223372036854775808) will be
returned.",
Review Comment:
`SystemFunctionUtils` (and `DateTimeUtils`) are evaluated inside Janino
engine. Will these error messages ever be printed out?
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
