Skip to content

unused import warning when building without the sql feature #18479

Description

@corasaurus-hex

Describe the bug

I was running commands from CI locally to ensure another PR was solid and I came across this warning:

❯ cargo check --profile ci --no-default-features -p datafusion
    Checking datafusion v50.3.0 (/Users/cora/Code/datafusion/datafusion/core)
warning: unused import: `Dialect`
  --> datafusion/core/src/execution/session_state.rs:40:65
   |
40 | use datafusion_common::config::{ConfigExtension, ConfigOptions, Dialect, TableOptions};
   |                                                                 ^^^^^^^
   |
   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default

warning: `datafusion` (lib) generated 1 warning (run `cargo fix --lib -p datafusion` to apply 1 suggestion)
    Finished `ci` profile [unoptimized] target(s) in 3.22s

To Reproduce

cargo check --profile ci --no-default-features -p datafusion

Expected behavior

No warnings

Additional context

Low priority, just a warning I noticed. I'll put up a PR momentarily.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions