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

Ballista integration tests fail to run (feature edition2021 is required) #1272

Closed
andygrove opened this issue Nov 8, 2021 · 2 comments · Fixed by #1274
Closed

Ballista integration tests fail to run (feature edition2021 is required) #1272

andygrove opened this issue Nov 8, 2021 · 2 comments · Fixed by #1274
Assignees
Labels
bug Something isn't working

Comments

@andygrove
Copy link
Member

Describe the bug

I tried running the Ballista integration tests from the master branch and from #1253 and I get the following error.

Step 22/56 : RUN cargo chef cook $RELEASE_FLAG --recipe-path recipe.json
 ---> Running in 874850134f2a
error: failed to load manifest for workspace member `/tmp/ballista/datafusion`

Caused by:
  failed to parse manifest at `/tmp/ballista/datafusion/Cargo.toml`

Caused by:
  feature `edition2021` is required

  this Cargo does not support nightly features, but if you
  switch to nightly channel you can add
  `cargo-features = ["edition2021"]` to enable this feature
thread 'main' panicked at 'Exited with status code: 101', /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/cargo-chef-0.1.23/src/recipe.rs:142:27
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
The command '/bin/sh -c cargo chef cook $RELEASE_FLAG --recipe-path recipe.json' returned a non-zero code: 101

To Reproduce
Run ./dev/integration-tests.sh

Expected behavior
Tests should pass

Additional context
None

@andygrove andygrove added the bug Something isn't working label Nov 8, 2021
@xudong963
Copy link
Member

xudong963 commented Nov 8, 2021

Hi @andygrove, after datafusion used edition 2021, I also encounter similar error information. So I think you can try to upgrade the rust version.

@andygrove
Copy link
Member Author

Thanks @xudong963. The Rust version is defined in the Dockerfile in the repo. I will create a PR to update it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants