Skip to content

[cosn] Support alternative COSN authentication options - #10130

Merged
JingsongLi merged 1 commit into
apache:masterfrom
thswlsqls:fix/support-alternative-cosn-authentication-options
Sep 24, 2026
Merged

JingsongLi merged 1 commit into
apache:masterfrom
thswlsqls:fix/support-alternative-cosn-authentication-options

Conversation

@thswlsqls

Copy link
Copy Markdown
Contributor

Purpose

fix #10129

Tests

  • COSNLoaderTest: endpoint suffix with default/custom provider configuration and region without static credentials; 3/3 passed.
  • Restoring only COSNLoader.java from upstream/master produced 3/3 errors; fixed HEAD passed 3/3.
  • mvn -pl paimon-filesystems/paimon-cosn -DfailIfNoTests=false clean install passed on JDK 11, including checkstyle, spotless, enforcer, and surefire.

Allow the loader to accept the endpoint suffix alternative and defer credential selection to hadoop-cos.

Generated-by: Codex
@JingsongLi

Copy link
Copy Markdown
Contributor

This addresses a real configuration failure: FileIO.get(cosn://...) previously rejected endpoint-suffix and provider-chain setups before the bundled hadoop-cos implementation could authenticate. The loader change is appropriately narrow, preserves region selection, and COSNFileIO.configure forwards fs.cosn.* options to Hadoop.

I applied the PR patch and ran COSNLoaderTest on both JDK 8 and JDK 11: 3/3 passed in each run. These tests verify plugin selection; they do not exercise actual environment/custom credential acquisition or an object-store operation. Please provide an integration or documented staging validation for the intended provider chain before production rollout. The current PR checks also include a failing JDK 11 Core and integrations lane (plus the aggregate result); establish the cause or get CI green before merge. The change has end-to-end value and I found no confirmed code defect in the loader.

@JingsongLi
JingsongLi merged commit 8be22d9 into apache:master Sep 24, 2026
16 of 18 checks passed
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.

[Bug] COSN loader rejects supported authentication and endpoint options

2 participants