buptljy commented on issue #6381: [FLINK-7205] [table] Add UUID supported in SQL and TableApi URL: https://github.com/apache/flink/pull/6381#issuecomment-410639702 @hequn8128 First, NULLIF(uuid(), uuid()) won't succeed to return null because of the "ReduceExpressionsRule", which will regard uuid() and uuid() as the same based on the "uuid()" string, it's the same for othere functions like current_timestamp. Second, putting two uuid() into the same row may not test "isDeterministic" function. for example, we assume that the "isDeterministic" returns true: the outputs are different when the input is "select uuid(),uuid() from testdata(1 row)", while the outputs are the same when the input is "select uuid() from testdata(2 rows)".
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
