Kontinuation commented on PR #12806:
URL: https://github.com/apache/iceberg/pull/12806#issuecomment-2818210775
I would suggest simply removing the assertion on the error message
`.hasMessageContaining("out of bounds");`. The exact error message of the
ArrayIndexOutOfBoundsException raised by indexing Java array is not guaranteed
by the Java specification, and the error messages raised by JVM may change over
time (see https://openjdk.org/jeps/358 as an example).
`.isInstanceOf(ArrayIndexOutOfBoundsException.class)` in the original test
is already a proper validation of expected behavior. Removing the assertion on
error message would make the test more robust against JVM implementation
differences.
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
