Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[minor]Fix ci clippy for unused import #2586

Merged
merged 1 commit into from
May 22, 2022
Merged

Conversation

Ted-Jiang
Copy link
Member

Which issue does this PR close?

Block current Pr

 Checking datafusion-common v8.0.0 (/__w/arrow-datafusion/arrow-datafusion/datafusion/common)
    Checking datafusion-expr v8.0.0 (/__w/arrow-datafusion/arrow-datafusion/datafusion/expr)
    Checking datafusion-row v8.0.0 (/__w/arrow-datafusion/arrow-datafusion/datafusion/row)
    Checking datafusion-physical-expr v8.0.0 (/__w/arrow-datafusion/arrow-datafusion/datafusion/physical-expr)
    Checking datafusion-jit v8.0.0 (/__w/arrow-datafusion/arrow-datafusion/datafusion/jit)
    Checking datafusion v8.0.0 (/__w/arrow-datafusion/arrow-datafusion/datafusion/core)
error: unused import: `union_with_alias`
  --> datafusion/core/src/sql/planner.rs:32:5
   |
32 |     union_with_alias, Column, CreateCatalog, CreateCatalogSchema,
   |     ^^^^^^^^^^^^^^^^
   |
   = note: `-D unused-imports` implied by `-D warnings`

error: could not compile `datafusion` due to previous error
warning: build failed, waiting for other jobs to finish...
error: could not compile `datafusion` due to previous error

Closes #.

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@github-actions github-actions bot added the datafusion Changes in the datafusion crate label May 22, 2022
@Ted-Jiang Ted-Jiang changed the title [minor]Fix ci clippy [minor]Fix ci clippy for unused import May 22, 2022
@Ted-Jiang
Copy link
Member Author

@alamb @andygrove FYI

@Ted-Jiang
Copy link
Member Author

Block #2580

@alamb
Copy link
Contributor

alamb commented May 22, 2022

I think this might be a dupe of #2585

@andygrove andygrove merged commit 844bcda into apache:master May 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
datafusion Changes in the datafusion crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants