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

   ### Purpose
   
   Linked issue: None
   
   Introduce the local file system implementation migrated from the Alibaba 
Paimon C++ repository.
   
   This PR adds:
   - local file primitives and file status wrappers
   - `LocalFileSystem` input/output stream support
   - local file system factory registration
   - local file unit coverage
   
   The requested migration scope was `src/paimon/fs/local` excluding 
`CMakeLists.txt`. No additional dependency files were required.
   
   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
   
   - `python3 
/home/jinli.zjw/.codex/skills/paimon-cpp-migrate/scripts/check_migration_batch.py
 --files src/paimon/fs/local/local_file_system_factory.h 
src/paimon/fs/local/local_file_test.cpp src/paimon/fs/local/local_file.h 
src/paimon/fs/local/local_file_system.cpp 
src/paimon/fs/local/local_file_system.h src/paimon/fs/local/local_file.cpp 
src/paimon/fs/local/local_file_system_factory.cpp 
src/paimon/fs/local/local_file_status.h`
   - `python3 
/home/jinli.zjw/.codex/skills/paimon-cpp-migrate/scripts/analyze_external_contributors.py
 --files src/paimon/fs/local/local_file_system_factory.h 
src/paimon/fs/local/local_file_test.cpp src/paimon/fs/local/local_file.h 
src/paimon/fs/local/local_file_system.cpp 
src/paimon/fs/local/local_file_system.h src/paimon/fs/local/local_file.cpp 
src/paimon/fs/local/local_file_system_factory.cpp 
src/paimon/fs/local/local_file_status.h`
   - `clang-format -i src/paimon/fs/local/local_file_system_factory.h 
src/paimon/fs/local/local_file_test.cpp src/paimon/fs/local/local_file.h 
src/paimon/fs/local/local_file_system.cpp 
src/paimon/fs/local/local_file_system.h src/paimon/fs/local/local_file.cpp 
src/paimon/fs/local/local_file_system_factory.cpp 
src/paimon/fs/local/local_file_status.h`
   - `git diff --cached --check`
   - `c++ -std=c++17 -Iinclude -Isrc -fPIC -c 
src/paimon/fs/local/local_file_system_factory.cpp -o 
/tmp/paimon_local_file_system_factory_final.o`
   
   Full local build was not run because the current Apache checkout does not 
have a root `CMakeLists.txt`, and the existing `build/` directory does not 
contain generated Makefile/Ninja files.
   
   ### API and Format
   
   This adds local file system implementation classes under 
`src/paimon/fs/local`. 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