Skip to content

Minor: Fix compiler warning when compiling datafusion-cli#17205

Merged
2010YOUY01 merged 2 commits into
apache:mainfrom
2010YOUY01:fix-not-used
Aug 15, 2025
Merged

Minor: Fix compiler warning when compiling datafusion-cli#17205
2010YOUY01 merged 2 commits into
apache:mainfrom
2010YOUY01:fix-not-used

Conversation

@2010YOUY01

@2010YOUY01 2010YOUY01 commented Aug 15, 2025

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

  • Closes #.

Rationale for this change

When compiling datafusion-cli with cargo run, there is a compiler warning:

warning: unused variable: `options`
   --> datafusion/datasource-parquet/src/reader.rs:247:9
    |
247 |         options: Option<&'a ArrowReaderOptions>,
    |         ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_options`
    |
    = note: `#[warn(unused_variables)]` on by default

This PR uses #[allow(unused_variables)] to suppress the warning.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions Bot added the datasource Changes to the datasource crate label Aug 15, 2025

@zhuqi-lucas zhuqi-lucas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Thank you @2010YOUY01 !

@2010YOUY01
2010YOUY01 merged commit b4c7ef7 into apache:main Aug 15, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

datasource Changes to the datasource crate

2 participants