Support pluggable AWS credentials for the S3 object store (credentials provider callback)
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Feature
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Bereich
- authentication, backend, cloud
Rechercherichtung
Beginne mit native/src/object_store.rs und dem S3-Build-Pfad und vergleiche anschließend das JNI-GlobalRef- und attach_current_thread-Muster in native/src/udf.rs. Verfolge, wie ObjectStoreRegistration- und UDF-Objekte JNI durchlaufen, und definiere den Abschluss als einen threadsicheren S3-Provider-Callback mit ablaufbewusstem Caching, Exklusivität statischer Zugangsdaten, Exception-Übersetzung und der Bereinigung von GlobalRef.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Java
- Sterne
- 32
- Forks
- 12
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus apache/datafusion-java
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
apache/datafusion-java#112 ·
-
enhancement
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 42/100
apache/datafusion-java#96 ·
-
enhancement
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 38/100
apache/datafusion-java#95 ·
-
Create first release Offenenhancement
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
apache/datafusion-java#86 · 3 Kommentare ·
-
enhancement
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 45/100
apache/datafusion-java#68 ·
Alle Issues in apache/datafusion-java
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
elastic/gradle-plugins#157 ·
-
enhancement Tools
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 75/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
apache/rocketmq-dashboard#5008 ·
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
DETECT_PARAMETER_NAMES=false silently disables @ConstructorProperties-based Creator detection too Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
FasterXML/jackson-databind#6229 ·