davidradl commented on code in PR #26924:
URL: https://github.com/apache/flink/pull/26924#discussion_r2303870692
##########
flink-table/flink-table-common/src/main/java/org/apache/flink/table/types/inference/SystemTypeInference.java:
##########
@@ -147,7 +159,9 @@ private static void
checkScalarArgsOnly(List<StaticArgument> defaultArgs) {
checkPassThroughColumns(declaredArgs);
final List<StaticArgument> newStaticArgs = new
ArrayList<>(declaredArgs);
- newStaticArgs.addAll(PROCESS_TABLE_FUNCTION_SYSTEM_ARGS);
Review Comment:
I am curious what the system arguments mean. Is this something that the user
needs to be aware of? I do not see this phrase in the Flip and there is no more
information in the Jira. I suggest including a description and motivation
behind this piece. It appears to be a type of static arg that will be added if
the boolean flag is on, but I am not sure when this would/should be used.
--
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]
