dramaticlly commented on issue #11554:
URL: https://github.com/apache/iceberg/issues/11554#issuecomment-2480067675
thank you @haizhou-zhao , I realized we actually have similar special
handling for IllegalArgumentException per #9225 and I am wondering if it's
worth doing the same for ValidationException.
The original reason I use VE instead of IAE is because it's already being
used in
https://github.com/apache/iceberg/blob/main/core/src/main/java/org/apache/iceberg/TableMetadata.java#L135
and
https://github.com/apache/iceberg/blob/main/core/src/main/java/org/apache/iceberg/MetricsConfig.java#L204-L213
I think 400 generally is correct behaviour as user need to change their
input, as of now the easier thing is probably add `assumeThat` so that this
particular test will only run against the hive/hadoop catalog
--
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]
