samueleresca opened a new issue, #17522:
URL: https://github.com/apache/datafusion/issues/17522
### Is your feature request related to a problem or challenge?
`tempfile` crate is currently included as a normal dependency in the
`datafusion` crate. It would be possible to move the dependency to the
`dev-dependencies` since the `tempfile` capabilities are only used by
integration and unit tests.
Specifically, the following function:
```
/// Generate CSV partitions within the supplied directory
pub fn populate_csv_partitions(
tmp_dir: &TempDir,
partition_count: usize,
file_extension: &str,
) -> Result<SchemaRef> {
...
```
It is the only one that uses `tempfile` capabilities.
### Describe the solution you'd like
It would be possible to move the `tempfile` crate from `dependencies` to
`dev-dependencies` by decorating the `populate_csv_partitions` with the
`#[cfg(test)]` attribute and replicating the function where needed in the
integration tests (i.e.: `datafusion/core/tests/dataframe/mod.rs`)
### Describe alternatives you've considered
_No response_
### Additional context
I'm opening this as a feature request since it's not an actual bug.
--
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.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
