Skip to content

Add more context to error message for datafusion-cli config failure #16379

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
Jun 11, 2025

Conversation

alamb
Copy link
Contributor

@alamb alamb commented Jun 11, 2025

Which issue does this PR close?

Rationale for this change

CI is failing, but when I ran the same command as is failing on CI locally, it passes for me:

cargo test  --profile ci --exclude datafusion-examples --exclude datafusion-benchmarks --exclude datafusion-sqllogictest --workspace --lib --tests --features=force_hash_collisions,avro

Therefore I conclude something about the environment on the github builder is causing the issue

What changes are included in this PR?

Add context from the inner error source as well to the error message, so I can debug what is the actual problem

Are these changes tested?

Are there any user-facing changes?

@blaginin
Copy link
Contributor

So strange! Passing for me locally as well, maybe some env variables..

@alamb alamb merged commit f8aad68 into apache:main Jun 11, 2025
28 checks passed
@alamb
Copy link
Contributor Author

alamb commented Jun 11, 2025

Let's get some data

1 similar comment
@alamb
Copy link
Contributor Author

alamb commented Jun 11, 2025

Let's get some data

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants