alamb commented on code in PR #16401: URL: https://github.com/apache/datafusion/pull/16401#discussion_r2155411540
########## datafusion/catalog/src/information_schema/tests.rs: ########## @@ -0,0 +1,88 @@ +use std::sync::Arc; Review Comment: The CI is failing because this file doesn't have the ASF header - Actually, it seems we were checking twice 🤦 https://github.com/apache/datafusion/pull/16451 I think it is more common that these types of tests go in the same module (file) at the bottom in the DataFusion codebase anyways so I'll move them there to fix the CI as well I'll move them there -- 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
