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

ambiguous glob re-exports of contains_utf8 #4289

Closed
xxchan opened this issue May 26, 2023 · 2 comments · Fixed by #4342
Closed

ambiguous glob re-exports of contains_utf8 #4289

xxchan opened this issue May 26, 2023 · 2 comments · Fixed by #4342
Assignees
Labels
arrow Changes to the arrow crate bug parquet Changes to the parquet crate

Comments

@xxchan
Copy link
Contributor

xxchan commented May 26, 2023

Describe the bug

warning: ambiguous glob re-exports
  --> arrow/src/compute/kernels/mod.rs:31:13
   |
31 |     pub use arrow_ord::comparison::*;
   |             ^^^^^^^^^^^^^^^^^^^^^^^^ the name `contains_utf8` in the value namespace is first re-exported here
32 |     pub use arrow_string::like::*;
   |             --------------------- but the name `contains_utf8` in the value namespace is also re-exported here
   |
   = note: `#[warn(ambiguous_glob_reexports)]` on by default

To Reproduce

cargo check

Expected behavior

Additional context

> rustc -vV                    
rustc 1.71.0-nightly (c4190f2d3 2023-05-07)
binary: rustc
commit-hash: c4190f2d3a46a59f435f7b42f58bc22b2f4d6917
commit-date: 2023-05-07
host: aarch64-apple-darwin
release: 1.71.0-nightly
LLVM version: 16.0.2
@xxchan xxchan added the bug label May 26, 2023
@tustvold tustvold self-assigned this Jun 2, 2023
tustvold added a commit to tustvold/arrow-rs that referenced this issue Jun 2, 2023
tustvold added a commit that referenced this issue Jun 2, 2023
* Rename list contains kernels to in_list (#4289)

* Fix other clippy lints
@tustvold tustvold added the parquet Changes to the parquet crate label Jun 2, 2023
@tustvold
Copy link
Contributor

tustvold commented Jun 2, 2023

label_issue.py automatically added labels {'parquet'} from #4342

@tustvold
Copy link
Contributor

tustvold commented Jun 2, 2023

label_issue.py automatically added labels {'arrow'} from #4342

@tustvold tustvold added the arrow Changes to the arrow crate label Jun 2, 2023
mcheshkov added a commit to cube-js/arrow-rs that referenced this issue Aug 27, 2024
Can drop this after rebase on commit 7952595 "Rename list contains kernels to in_list (apache#4289) (apache#4342)", first released in 41.0.0
mcheshkov added a commit to cube-js/arrow-rs that referenced this issue Aug 27, 2024
Can drop this after rebase on commit 7952595 "Rename list contains kernels to in_list (apache#4289) (apache#4342)", first released in 41.0.0
mcheshkov added a commit to cube-js/arrow-rs that referenced this issue Aug 27, 2024
Can drop this after rebase on commit 7952595 "Rename list contains kernels to in_list (apache#4289) (apache#4342)", first released in 41.0.0
mcheshkov added a commit to cube-js/arrow-rs that referenced this issue Aug 27, 2024
Can drop this after rebase on commit 7952595 "Rename list contains kernels to in_list (apache#4289) (apache#4342)", first released in 41.0.0
mcheshkov added a commit to cube-js/arrow-rs that referenced this issue Sep 5, 2024
Can drop this after rebase on commit 7952595 "Rename list contains kernels to in_list (apache#4289) (apache#4342)", first released in 41.0.0
mcheshkov added a commit to cube-js/arrow-rs that referenced this issue Sep 5, 2024
Can drop this after rebase on commit 7952595 "Rename list contains kernels to in_list (apache#4289) (apache#4342)", first released in 41.0.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arrow Changes to the arrow crate bug parquet Changes to the parquet crate
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants