alan910127 commented on code in PR #15404: URL: https://github.com/apache/datafusion/pull/15404#discussion_r2011249477
########## datafusion/sqllogictest/test_files/information_schema.slt: ########## @@ -736,8 +736,11 @@ select specific_name, data_type, ordinal_position, parameter_mode, rid from info repeat Int64 2 IN 0 repeat Int64 2 IN 1 repeat Int64 2 IN 2 +repeat Int64 2 IN 3 repeat LargeUtf8 1 IN 1 repeat LargeUtf8 1 OUT 1 +repeat Null 1 IN 3 +repeat Null 1 OUT 3 Review Comment: I'm not sure why this was changed, as there seems to already be a test case for `SELECT repeat(NULL, 4)` in the original version, so the information schema shouldn't need to be modified? -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
