JingsongLi commented on PR #10899:
URL: https://github.com/apache/paimon/pull/10899#issuecomment-6557124396

   The one-line fixture change is correct for its two callers: both patched 
files are direct children of the JUnit temporary directory, and their names 
identify the same files written by `writeRowFile`. Corrupted bytes and 
assertions are unchanged.
   
   This only makes two existing corruption tests runnable on Windows. 
Production ROW validation/read code and persisted data are unchanged, and the 
current Java CI runs on Ubuntu. Closing the standalone fixture portability PR 
under the current end-to-end production contribution scope; it can accompany a 
concrete supported-platform or functional change.
   
   Verification: all 9 `RowFileIndexConsistencyTest` cases pass on JDK 8/macOS 
with normal Maven `verify`, including Checkstyle, Spotless, Enforcer and RAT; 
CI is green. Windows was not independently run here. No implementation defect 
found.
   


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