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

   ### Purpose
   
   This is a narrower follow-up to #6653 for engines such as Trino that provide 
their own FileIO and should not require Hadoop configuration initialization.
   
   Instead of refactoring the FileIO/CatalogContext hierarchy, this keeps all 
existing `CatalogContext.create(...)` behavior unchanged and adds an explicit 
`CatalogContext.createWithoutHadoop(...)` factory for the no-Hadoop path.
   
   ### Changes
   
   - Keep existing `CatalogContext.create(...)` overloads loading Hadoop 
configuration by default for compatibility.
   - Add `CatalogContext.createWithoutHadoop(...)` for callers that provide 
their own `FileIOLoader`.
   - Make `hadoopConf()` fail with a clear `IllegalStateException` when called 
on a Hadoop-free context.
   - Add catalog tests, including a classloader test that filters Hadoop 
classes and verifies the new path can create a catalog without Hadoop on the 
classpath.
   
   ### Tests
   
   - `mvn -pl paimon-core -am -Pfast-build -DfailIfNoTests=false 
-Dtest=CatalogFactoryTest test`
   - `mvn -pl paimon-common -am -Pfast-build -DfailIfNoTests=false 
-Dtest=FileIOTest,ResolvingFileIOTest test`
   
   ### Notes
   
   A companion Trino change is being prepared to consume this API by passing 
Trino's FileIO loader and disabling Hadoop default configuration loading.


-- 
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]

Reply via email to