zjw1111 opened a new pull request, #42:
URL: https://github.com/apache/paimon-cpp/pull/42

   ### Purpose
   
   Linked issue: None
   
   Introduce file system test coverage migrated from the Alibaba Paimon C++ 
repository.
   
   This PR adds:
   - `FileSystem` unit coverage
   - `ResolvingFileSystem` unit coverage
   - `ExternalPathProvider` and its unit coverage
   
   The requested migration scope is limited to the test files under 
`src/paimon/common/fs` plus `external_path_provider.h`. Local file system 
implementation dependencies are intentionally kept out of this PR and are 
covered by #41.
   
   License handling:
   - Replaced Alibaba-owned source headers with ASF headers.
   - Checked source and target `LICENSE` / `NOTICE`; no third-party declaration 
updates were required for these files.
   
   External contributor handling:
   - Re-ran blame-based contributor analysis for the final file set.
   - No external contributor thresholds were hit, so no `Co-authored-by` 
trailer or PR thank-you comment is needed.
   
   ### Tests
   
   - `git diff --check`
   
   ### API and Format
   
   This adds `ExternalPathProvider` under `src/paimon/common/fs` for external 
data path selection. It does not change public storage format or protocol.
   
   ### Documentation
   
   No documentation changes.
   
   ### Generative AI tooling
   
   Migrated-by: OpenAI Codex
   


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