dylzlt commented on pull request #4258:
URL: https://github.com/apache/iceberg/pull/4258#issuecomment-1064794777
> @dylzlt sorry for the delay, I synced with @RussellSpitzer to reduce the
back and forth, his main concern was to make sure we say
IllegalArgumentException means it is never valid as opposed to it's not valid
now because of other arguments.
>
> So can you do something like?
>
> ```
> Exception which is raised when the arguments are valid in isolation, but
not in conjunction with other arguments or state, as opposed to {@link
IllegalArgumentException} which is raised when an argument value is always
invalid. A ValidationException will cause the operation to abort.
> ```
>
> I think it should be good to go after this.
ok, I get it. Thank you for your patience. I complete it.
--
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]
