Skip to content

fix: build Unity Catalog crate without DataFusion #3420

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

Merged
merged 1 commit into from
May 7, 2025

Conversation

linhr
Copy link
Contributor

@linhr linhr commented May 7, 2025

Description

I was trying to build the deltalake-catalog-unity crate without the datafusion feature. Here is the command I used:

cargo build -p deltalake-catalog-unity --no-default-features --features "aws"

I encountered the following error.

error[E0432]: unresolved import `datafusion_common`
  --> crates/catalog-unity/src/lib.rs:10:5
   |
10 | use datafusion_common::DataFusionError;
   |     ^^^^^^^^^^^^^^^^^ use of unresolved module or unlinked crate `datafusion_common`
   |
   = help: if you wanted to use a crate named `datafusion_common`, use `cargo add datafusion_common` to add it to your `Cargo.toml`

For more information about this error, try `rustc --explain E0432`.
error: could not compile `deltalake-catalog-unity` (lib) due to 1 previous error

This PR fixes the build error.

Related Issue(s)

N/A

Documentation

N/A

@github-actions github-actions bot added the binding/rust Issues for the Rust crate label May 7, 2025
@linhr linhr force-pushed the unity-catalog-feature-fix branch from b58ded1 to 905363b Compare May 7, 2025 07:34
@ion-elgreco ion-elgreco added this pull request to the merge queue May 7, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 7, 2025
@rtyler rtyler added this pull request to the merge queue May 7, 2025
Merged via the queue into delta-io:main with commit 696171a May 7, 2025
23 of 24 checks passed
@linhr linhr deleted the unity-catalog-feature-fix branch May 7, 2025 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binding/rust Issues for the Rust crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants