leonardBang commented on a change in pull request #15303:
URL: https://github.com/apache/flink/pull/15303#discussion_r598691545
##########
File path:
flink-table/flink-table-planner-blink/src/main/scala/org/apache/flink/table/planner/codegen/CodeGeneratorContext.scala
##########
@@ -505,8 +598,27 @@ class CodeGeneratorContext(val tableConfig: TableConfig) {
}
/**
- * Adds a reusable local time to the beginning of the SAM of the generated
class.
- */
+ * Adds a reusable session-level local date time to the beginning of
+ * the SAM of the generated class.
+ */
+ def addReusableSessionLocalDateTime(): String = {
Review comment:
You're right, @twalthr , we need to consider the session time should be
used in the entire plan.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
