jackye1995 commented on a change in pull request #3175:
URL: https://github.com/apache/iceberg/pull/3175#discussion_r717276851



##########
File path: api/src/main/java/org/apache/iceberg/io/OutputFile.java
##########
@@ -48,6 +49,7 @@
    *
    * @return an output stream that can report its position
    * @throws RuntimeIOException If the implementation throws an {@link 
IOException}
+   * @throws AccessControlException If the implementation throws an {@link 
SecurityException}

Review comment:
       as discussed in the block below, we can just document @throws 
SecurityException if staging directory creation fails due to missing JVM level 
permission.




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