wiedld commented on code in PR #14637: URL: https://github.com/apache/datafusion/pull/14637#discussion_r1966344294
########## datafusion/sqllogictest/test_files/union_by_name.slt: ########## @@ -244,13 +244,14 @@ SELECT x, y FROM t1 UNION BY NAME (SELECT y, z FROM t2 INTERSECT SELECT 2, 2 as 3 3 NULL NULL 2 2 -query III +# TODO: Test regression on a new feature, with the same SanityCheckPlan failures as currently existing for this feature. +query error SELECT x, y FROM t1 UNION ALL BY NAME (SELECT y, z FROM t2 INTERSECT SELECT 2, 2 as two FROM t1 ORDER BY 1) ORDER BY 1; ---- -1 1 NULL -3 3 NULL -3 3 NULL -NULL 2 2 +DataFusion error: SanityCheckPlan +caused by Review Comment: In this file, there are 3 other SanityCheckPlan failures in the same (newly added) union-by-name feature. I'm working on a fix for the union-by-name. So IMO this is not a blocker. Feedback? -- 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
