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

Run _init_credential_provider before executing deserialized logical plan if credential_provider="auto" #21157

Closed
nameexhaustion opened this issue Feb 10, 2025 · 0 comments · Fixed by #21225
Assignees
Labels
A-io-cloud Area: reading/writing to cloud storage A-serde Area: seralization and deserialization accepted Ready for implementation enhancement New feature or an improvement of an existing feature P-high Priority: high

Comments

@nameexhaustion
Copy link
Collaborator

Description

This currently doesn't run and instead the credentials from the system that serialized the plan is used instead (or no provider is used, if no credential provider ends up getting serialized). This can break e.g. if a query plan is created on a local system that has differing credentials from the remote system that actually executes the plan.

@nameexhaustion nameexhaustion added A-io-cloud Area: reading/writing to cloud storage A-serde Area: seralization and deserialization accepted Ready for implementation enhancement New feature or an improvement of an existing feature P-high Priority: high labels Feb 10, 2025
@nameexhaustion nameexhaustion self-assigned this Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-io-cloud Area: reading/writing to cloud storage A-serde Area: seralization and deserialization accepted Ready for implementation enhancement New feature or an improvement of an existing feature P-high Priority: high
Projects
None yet
1 participant