imbajin opened a new pull request, #777:
URL: https://github.com/apache/hugegraph-toolchain/pull/777

   ## Purpose of the PR
   
   HDFS restore previously attempted ZIP decoding for plain backup files. Read 
plain files directly and unzip only files with the .zip suffix, preserving 
uncompressed backups written by older Tools versions.
   
   Original preparation and review: 
https://github.com/hugegraph/hugegraph-toolchain/pull/42.
   
   Depends on https://github.com/apache/hugegraph-toolchain/pull/775. Merge the 
shared build preparation first. This PR targets master and currently includes 
prerequisite changes; after the prerequisite lands, rebase this same 
organization branch onto the updated upstream base and revalidate the module 
diff before merging.
   
   ## Main Changes
   
   - Support plain and ZIP input streams and close invalid compressed streams 
before reporting failure.
   - Exercise backup/restore through Hadoop's local file URI, including UTF-8 
data and compression suffix assertions.
   - Require explicit tools.test.disposable=true before destructive graph test 
setup; CI opts in only for its job-owned instance.
   - Complete independent cleanup actions without masking the primary failure.
   
   ## Verifying these changes
   
   - [ ] Trivial rework / code cleanup without any test coverage.
   - [x] Already covered by Hadoop stream tests and backup/restore regression 
tests.
   - [x] Need tests and can be verified as follows:
     - Prior functional validation used a dedicated released Server 1.7 
instance.
     - The latest stream and disposable-instance guard tests passed on native 
Java 17 without a backend.
     - Latest CLI round-trip validation is awaiting the organization CI job on 
its disposable server; it has not yet been recorded as passed.
   
   ## Does this PR potentially affect the following parts?
   
   - [ ] Nope
   - [ ] Dependencies (build/test tooling only; runtime payload is unchanged by 
this preparation)
   - [x] Modify configurations (CI/test configuration)
   - [ ] The public API (compatible behavior; signatures unchanged)
   - [x] Other affects: tests and validation tooling
   
   ## Documentation Status
   
   - [ ] Doc - TODO
   - [x] Doc - Done
   - [ ] Doc - No Need
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to