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

   ## Which issue does this PR close?
   - related to https://github.com/apache/datafusion/issues/11210
   
   
   ## Rationale for this change
   
   Running the following command
   
   ```shell
   cargo test --test csv_schema_fix_test
   ```
   
   Results in a 201 mb binary (and requires linking, etc) for only a few tests
   
   ```shell
   du -s -h target/debug/deps/csv_schema_fix_test-48507d40c6598e82
   201M target/debug/deps/csv_schema_fix_test-48507d40c6598e82
   ```
   
   
   Let's combine that into the existing core integration test to save CI runner 
space and time
   
   Also, I have been confused why the parquet integration tests are named 
`parquet_source` (I think it is meant to mirror  `ParquetSource` which is 
confusing) so let's change that to something more discoverable
   
   ## What changes are included in this PR?
   1. Consolidate core_integration/datasource
   2. rename `parquet_source` --> `parquet_integration`
   
   ## Are these changes tested?
   
   It is only tests. You can run the tests in the new location via
   ```shell
   cargo test --test core_integration -- csv
   ```
   
   ## Are there any user-facing changes?
   No, this is all internal test rearrangement
   


-- 
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