zzwqqq commented on code in PR #4942:
URL: https://github.com/apache/calcite/pull/4942#discussion_r3300922141
##########
core/src/main/java/org/apache/calcite/rex/RexBuilder.java:
##########
@@ -958,6 +958,11 @@ public RexNode makeCast(
return temporalLiteral;
}
+ /** Converts a TIME or TIMESTAMP literal to a character literal.
+ *
+ * <p>Returns null if the literal type is not supported, the literal value is
+ * unavailable, or the formatted temporal value does not fit in the target
Review Comment:
Thanks, I removed the caller behavior from the JavaDoc and made the
null-return cases more specific.
--
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]