szehon-ho commented on a change in pull request #4258:
URL: https://github.com/apache/iceberg/pull/4258#discussion_r824416909
##########
File path:
api/src/main/java/org/apache/iceberg/exceptions/ValidationException.java
##########
@@ -24,7 +24,11 @@
import org.apache.iceberg.Schema;
/**
- * Exception raised when validation checks fail.
+ * Exception which is raised when the arguments are valid in isolation,
Review comment:
Small nit, can you try to make each line be longer and try to line up
towards the right?
I don't see many places where the doc line-breaks on a comma, usually it's
on a period. You can also add a <p> after the period.
A good example in my opinion :
https://github.com/apache/iceberg/blob/master/api/src/main/java/org/apache/iceberg/ExpireSnapshots.java
Thank you
--
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]
