adamreeve commented on code in PR #17103:
URL: https://github.com/apache/datafusion/pull/17103#discussion_r2265459596
##########
datafusion/common/src/config.rs:
##########
@@ -2626,7 +2665,7 @@ impl Display for OutputFormat {
mod tests {
use crate::config::{
ConfigEntry, ConfigExtension, ConfigField, ConfigFileType,
ExtensionOptions,
- Extensions, TableOptions,
+ Extensions, TableOptions, TableParquetOptions,
Review Comment:
This line is what has caused the build failure. Because you've imported the
`TableParquetOptions` name, places where it is already used in the `tests`
module in fully qualified form (`crate::config::TableParquetOptions`) now raise
an error.
You can reproduce it if you run the same command as CI:
```
cargo check --profile ci --workspace --all-targets --features
integration-tests --locked
```
To fix it, you can either revert this change and refer to it as
`crate::config::TableParquetOptions` in the new test, or make the change
suggested in the error.
--
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
