You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "thswlsqls (via GitHub)" <gi...@apache.org> on 2026/07/31 05:05:34 UTC

[PR] [filesystem][obs] Remove access key and secret from required options in OBSLoader [paimon]

thswlsqls opened a new pull request, #8958:
URL: https://github.com/apache/paimon/pull/8958

   ### Purpose
   
   fix #8957
   
   - Required options are a selection gate: declaring `fs.obs.access.key`/`fs.obs.secret.key` makes `FileIO.get()` discard `OBSLoader` for catalogs without static credentials.
   - The `HadoopFileIO` fallback cannot resolve `OBSFileSystem` (plugin-only classloader), so `obs://` fails with `UnsupportedSchemeException`.
   - hadoop-huaweicloud authenticates without static keys: `DefaultOBSClientFactory` builds an `ObsClient` from the `fs.obs.security.provider` `IObsCredentialsProvider` when both are empty.
   - Mirrors #8679, which removed the same gate from `S3Loader`.
   - `fs.obs.endpoint` stays required — `DefaultOBSClientFactory` reads it without a default.
   
   ### Tests
   
   - Added `OBSLoaderTest#testScheme` and `#testDoesNotRequireCredentials`, asserting required options flatten to exactly `fs.obs.endpoint`.
   - `mvn -pl paimon-filesystems/paimon-obs -DfailIfNoTests=false clean install` on JDK 11: 2 tests passed.
   - No end-to-end test against a real OBS backend — the build has no OBS emulator. Coverage is the loader's required-options contract only.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [filesystem][obs] Remove access key and secret from required options in OBSLoader [paimon]

Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on PR #8958:
URL: https://github.com/apache/paimon/pull/8958#issuecomment-5165548272

   +1


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [filesystem][obs] Remove access key and secret from required options in OBSLoader [paimon]

Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #8958:
URL: https://github.com/apache/paimon/pull/8958


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org