alamb opened a new pull request, #14244: URL: https://github.com/apache/datafusion/pull/14244
Move tests ## Which issue does this PR close? - Follow on to https://github.com/apache/datafusion/pull/14190 Part of - https://github.com/apache/datafusion/issues/11502 ## Rationale for this change Now that we have migrated the physical optimizer tests over, @berkaysynnada and @logan-keede made the modules smaller and easier to manage by putting the tests into separate integration test Let's keep the rest of the tests following the same pattern and consolidate them with the existing files ## What changes are included in this PR? 1. Consolidate the tests into a single binary: `physical_optimizer_integration`, which is consistent with `optimizer_integration`, `sql_integration`, etc 2. Extract tests from other large optimizer modules and put them into test modules 3. Add comments to point the way ## Are these changes tested? Yes by CI. You can run them via ```shell cargo test --test core_integration ``` ## Are there any user-facing changes? <!-- If there are user-facing changes then we may require documentation to be updated before approving the PR. --> <!-- If there are any breaking changes to public APIs, please add the `api change` label. --> -- 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
