kerwin-zk opened a new pull request, #8706: URL: https://github.com/apache/paimon/pull/8706
### Purpose When PyPaimon reads an OSS-backed table through Daft's native parquet reader, the catalog options are converted to a Daft IOConfig. Today the OSS branch only fills the OpenDAL OSS backend. That works for Daft builds that route oss:// through OpenDAL, but Daft builds with a Jindo backend need the same fs.oss.* credentials in IOConfig.jindo. This patch keeps the existing OpenDAL OSS config and, when the installed Daft exposes JindoConfig, also fills IOConfig.jindo from the same fs.oss.* catalog options. Older Daft builds without JindoConfig keep the previous IOConfig shape. ### Tests - python -m pytest paimon-python/pypaimon/tests/daft/daft_io_config_test.py paimon-python/pypaimon/tests/daft/daft_blob_test.py paimon-python/pypaimon/tests/daft/daft_data_test.py -q - python -m pytest paimon-python/pypaimon/tests/daft --ignore=paimon-python/pypaimon/tests/daft/daft_catalog_rest_test.py -q - Verified against an OSS-backed DLF REST table with Daft native parquet reader: source format parquet, Jindo config present, row count 1000. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
