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

   ### Purpose
   
   fix #8556
   
   - `HadoopCompliantFileIO.HadoopFileStatus` in `paimon-gs-impl` did not 
override `getAccessTime()`/`getOwner()`, so on GCS file status always returned 
the `FileStatus` interface defaults (`0` and `null`) instead of the values 
carried by the wrapped Hadoop `FileStatus`.
   - Add the two delegating overrides, restoring parity with the 
s3/oss/obs/cosn/azure impls, which already do this.
   - The delegation originated in core `HadoopFileIO`+s3+oss (#4459) and was 
added per module for the others; the later GS impl (#5238) omitted it.
   
   ### Tests
   
   - No unit test added: `HadoopFileStatus` is a private static delegation 
wrapper reachable only through a live GCS `FileSystem`; the five sibling impls 
merged the identical overrides without one.
   - `mvn -pl paimon-filesystems/paimon-gs-impl clean install` passed (compile 
+ checkstyle + spotless + rat).
   


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