Support pluggable AWS credentials for the S3 object store (credentials provider callback)
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Clairement spécifiée
- Activité
- Active
- Domaine
- authentication, backend, cloud
Piste de recherche
Commencez par native/src/object_store.rs et le chemin de build S3, puis comparez le modèle JNI GlobalRef et attach_current_thread dans native/src/udf.rs. Suivez le passage des objets ObjectStoreRegistration et UDF à travers JNI, et définissez l'achèvement comme un callback de fournisseur S3 thread-safe avec mise en cache tenant compte de l'expiration, exclusivité des identifiants statiques, traduction des exceptions et nettoyage de GlobalRef.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Is your feature request related to a problem or challenge?
ObjectStoreOptions.S3 currently supports two credential modes: static values (accessKeyId / secretAccessKey / sessionToken) passed once at registration, or falling through to object_store's built-in provider chain (IMDS instance profile, ECS task endpoint, EKS pod identity, web identity).
For a long-lived SessionContext — the intended usage pattern for DataFusion — this leaves no working credential story in several common deployments:
- STS-assumed roles: IMDS only serves the instance-profile role. Credentials for an assumed role (cross-account, scoped-down, role chaining) can only be passed statically, and expire after 1–12h with no refresh. Queries then start failing with
ExpiredTokenuntil the store is somehow rebuilt. - AWS Lambda: there is no IMDS endpoint. Credentials arrive via environment variables, which
object_storecaptures once as aStaticCredentialProvider. ASessionContextcached across warm invocations goes stale when the environment credentials rotate underneath it. - SSO / profiles /
credential_process/ custom credential vendors: not inobject_store's chain at all.
Upstream has explicitly declined to widen the built-in chain: apache/arrow-rs#5143 (use the official AWS SDK) and apache/arrow-rs-object-store#47 (credential_process support) were both closed as not planned. The sanctioned extension point is
AmazonS3Builder::with_credentials(AwsCredentialProvider). Quoting the object_store maintainer in those threads:
"Users can provide their own credential providers […] This could even be using the AWS SDK if they so wish."
"Encouraging the downstreams to expose, or otherwise utilize the object_store credential provider API would avoid this entirely."
"I'd naturally prefer the solution that gives users the most flexibility and avoids needing to revisit this again when someone comes along requesting SSO or similar."
The Java binding does not expose this extension point, so Java users are locked out of everything the AWS SDK for Java's default credential chain already handles. There is direct precedent for fixing this at the binding layer: arrow-rs-object-store#47 was closed because polars exposed a credential provider
callback in its Python API ("Polars exposed it as part of their API and configured it to use boto when available. I'm going to close this…").
Describe the solution you'd like
An S3-specific credentials provider callback on the S3 options builder, bridged to AmazonS3Builder::with_credentials:
/**
* Supplies current AWS credentials. Invoked by the native layer when its cached
* credentials approach expiry; implementations must be thread-safe.
*/
@FunctionalInterface
public interface AwsCredentialsProvider {
AwsCredentials get();
}
public record AwsCredentials(
String accessKeyId,
String secretAccessKey,
Optional<String> sessionToken,
Optional<Instant> expiration) {}
SessionContext.builder()
.registerObjectStore(
ObjectStoreOptions.s3()
.bucket("my-bucket")
.credentialsProvider(provider) // new
.build())
.build();
// Typical usage delegates to the AWS SDK for Java, which brings the full default chain (SSO, profiles, credential_process, STS auto-refresh, Lambda env) for free:
var sdk = software.amazon.awssdk.auth.credentials.DefaultCredentialsProvider.create();
var options = ObjectStoreOptions.s3()
.bucket("my-bucket")
.credentialsProvider(() -> AwsCredentials.from(sdk.resolveCredentials()))
.build();
Native side. A Rust struct holding a JNI GlobalRef to the Java provider implements object_store::CredentialProvider<Credential = AwsCredential> and is passed to AmazonS3Builder::with_credentials in build_s3 (native/src/object_store.rs). The bridge caches the returned credentials keyed on expiration (no expiration
= cache indefinitely) with a min-TTL refresh-ahead check, so the JVM upcall fires roughly once per credential rotation, not per request. The repo already has the required upcall machinery: native/src/udf.rs holds a GlobalRef plus cached bridge class and calls attach_current_thread() to invoke Java from DataFusion
execution threads — the credentials bridge follows the same pattern. (object_store's own TokenCache is pub(crate), so the bridge implements its own small equivalent, as delta-rs does.)
Since registration currently crosses JNI as a serialized protobuf ObjectStoreRegistration, the provider jobject would be passed alongside the message rather than inside it — the same way UDF registration passes the UDF object.
Semantics.
- The provider is called on native (tokio worker) threads; implementations must be thread-safe.
- Returning an expiration opts into refresh: the native layer re-invokes the provider before expiry. Because every request resolves credentials at sign time, rotation is safe mid-query — long scans and multipart uploads spanning a rotation just sign later requests with the newer credentials.
- Mutually exclusive with static accessKeyId/secretAccessKey on the same builder — setting both is a build-time error.
- Java exceptions from the provider are translated into object_store::Error::Generic and surface as query errors.
- The GlobalRef is released when the registered store is dropped.
Non-goals.
- No generic cross-backend credential abstraction: with_credentials is typed per backend (AwsCredential vs GcpCredential bearer tokens), so this API belongs on the S3 builder only.
- GCS is out of scope: service-account keys and the metadata server already self-refresh in object_store. A gcs().credentialsProvider(...) (bearer-token supplier) can follow the same pattern later if a concrete need appears.
- No replication of the AWS credential chain inside the binding — resolving credentials is the AWS SDK for Java's job; the binding only ferries the result. The binding takes no dependency on the AWS SDK; a convenience adapter could be a follow-up.
Describe alternatives you've considered
- Do nothing / document the built-in chain. Leaves STS-assumed roles, Lambda with a cached context, SSO, and
credential_processbroken; upstream has frozen the built-in chain (apache/arrow-rs#5143, apache/arrow-rs-object-store#47), so this gap is permanent. - Imperative
refreshCredentials(newCreds)method. Requires the application to run its own expiry timers, is not atomic with respect to in-flight queries without extra locking in the binding, and puts lifecycle behavior on an immutable options object. The pull-based provider + cache is race-free by
construction (each request resolves current credentials) and matches the design every AWS SDK converged on. - Re-register the object store with new options. Same timer problem, plus undefined behavior for scans already planned against the previously registered store.
Additional context
- Maintainer guidance to downstream bindings: apache/arrow-rs#5143 (comments from 2024-08-29 onward) and apache/arrow-rs-object-store#47.
- Precedent: polars exposed exactly this callback in its Python API, which is what closed arrow-rs-object-store#47 (pola-rs/polars#18979).
- Rust-side reference implementation gluing the AWS SDK to a
CredentialProvider: delta-rscredentials.rs. - Existing Rust→Java upcall pattern in this repo:
native/src/udf.rs(GlobalRef+attach_current_thread).
I'm happy to implement this if the approach is agreed.
- Langage dominant
- Java
- Étoiles
- 32
- Forks
- 12
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de apache/datafusion-java
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
apache/datafusion-java#112 ·
-
enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 42/100
apache/datafusion-java#96 ·
-
enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 38/100
apache/datafusion-java#95 ·
-
Create first release Ouverteenhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
apache/datafusion-java#86 · 3 commentaires ·
-
enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 45/100
apache/datafusion-java#68 ·
Toutes les issues de apache/datafusion-java
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
elastic/gradle-plugins#157 ·
-
enhancement Tools
Difficulté 1/5 Moins d'une heure Accessibilité débutants 75/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
apache/rocketmq-dashboard#5008 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
DETECT_PARAMETER_NAMES=false silently disables @ConstructorProperties-based Creator detection too Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
FasterXML/jackson-databind#6229 ·