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

   ### Purpose
   
   fix #8953
   
   - `HadoopCompliantFileIO.HadoopFileStatus` in paimon-jindo did not override 
`getAccessTime()`/`getOwner()`, so the `FileStatus` defaults leaked through: 
access time `0`, owner `null`.
   - Delegate both to the wrapped `org.apache.hadoop.fs.FileStatus`, which 
already holds the real values.
   - `ObjectTableImpl.toRow()` fills the `atime`/`owner` columns from these 
getters, so `oss://` object tables showed `0`/`NULL`.
   - Every other Hadoop-compliant filesystem impl (s3/oss/obs/cosn/azure/gs) 
already delegates; paimon-jindo was the last one missing it — same change as 
#8559 for gs.
   
   ### Tests
   
   - No test added: `HadoopFileStatus` is `private static` and its enclosing 
class cannot be instantiated without a Jindo backend (`JindoHadoopSystem` loads 
a Linux x86_64 native library in its static initializer). Sibling #8559 also 
merged without tests.
   - `mvn -pl paimon-filesystems/paimon-jindo -DfailIfNoTests=false clean 
install` — BUILD SUCCESS, 19 existing tests passed.
   
   
   


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