matriv commented on a change in pull request #18624:
URL: https://github.com/apache/flink/pull/18624#discussion_r800722954
##########
File path:
flink-table/flink-table-planner/src/main/java/org/apache/flink/table/planner/utils/InternalConfigOptions.java
##########
@@ -50,4 +51,13 @@
+ " as UTC+0 milliseconds since epoch for
simplification, this config will be used by"
+ " some temporal functions like
LOCAL_TIMESTAMP in batch job to make sure these"
+ " temporal functions has query-start
semantics.");
+
+ @Experimental
+ public static final ConfigOption<Boolean>
TABLE_EXEC_NON_TEMPORAL_SORT_ENABLED =
+ key("table.exec.sort.non-temporal.enabled")
Review comment:
Ok, so it's ok to move it to the InternalOptions and the test usage is
still valid.
--
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
