alamb opened a new pull request, #11831:
URL: https://github.com/apache/datafusion/pull/11831

   ## Which issue does this PR close?
   N/A
   
   ## Rationale for this change
   
   Each individual .rs file in the `tests` directory results in a new test 
binary which means:
   * 10s of MB of new binary (and thus slower)
   * It can't run in parallel with other tests (only tests within the same 
binary do)
   
   ## What changes are included in this PR?
   
   Consolidate the `path_partition` test into the `core_integration` test to 
make testing more efficient
   
   ## Are these changes tested?
   By CI. I also tested locally
   
   You can see the tests run like this:
   
   ```shell
   $ cargo test --test core_integration -- path_partition
      Compiling datafusion-physical-optimizer v40.0.0 
(/Users/andrewlamb/Software/datafusion/datafusion/physical-optimizer)
      Compiling datafusion v40.0.0 
(/Users/andrewlamb/Software/datafusion/datafusion/core)
       Finished `test` profile [unoptimized + debuginfo] target(s) in 3.92s
        Running tests/core_integration.rs 
(target/debug/deps/core_integration-c1d878777c00ea56)
   
   running 9 tests
   test sql::path_partition::parquet_overlapping_columns ... ok
   test sql::path_partition::parquet_statistics ... ok
   test sql::path_partition::parquet_multiple_partitions ... ok
   test sql::path_partition::parquet_multiple_nonstring_partitions ... ok
   test sql::path_partition::csv_projection_on_partition ... ok
   test sql::path_partition::csv_filter_with_file_nonstring_col ... ok
   test sql::path_partition::csv_filter_with_file_col ... ok
   test sql::path_partition::csv_grouping_by_partition ... ok
   test sql::path_partition::parquet_distinct_partition_col ... ok
   
   test result: ok. 9 passed; 0 failed; 0 ignored; 0 measured; 200 filtered 
out; finished in 0.07s
   ```
   
   ## 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to