Aireed opened a new issue, #7285:
URL: https://github.com/apache/iceberg/issues/7285

   HadoopTableOperations#refresh call findVersion() to get the version of 
metadata.
   When we have no privilege to access the directory , the findVersion returns 
0 and  the  refresh method return null with information lost.
   
   i think we should catch the exception and throw it to the caller.
   
   
![image](https://user-images.githubusercontent.com/8862395/230262692-b7c8e215-e0e0-4d8e-b870-6bc094f2de12.png)
   
   
   
   
https://github.com/apache/iceberg/blob/7c61537194f9ab68e3d83a16b339ec47180f5f10/core/src/main/java/org/apache/iceberg/hadoop/HadoopTableOperations.java#L313
   
   > `int findVersion() {
       Path versionHintFile = versionHintFile();
       FileSystem fs = getFileSystem(versionHintFile, conf);
       try (InputStreamReader fsr =
               new InputStreamReader(**fs.open(versionHintFile)**, 
StandardCharsets.UTF_8);
           BufferedReader in = new BufferedReader(fsr)) {
         return Integer.parseInt(in.readLine().replace("\n", ""));
       } catch (Exception e) {


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