cvr-aditya commented on code in PR #18220:
URL: https://github.com/apache/iceberg/pull/18220#discussion_r4192417283
##########
spark/v4.1/spark/src/test/java/org/apache/iceberg/spark/sql/TestTableEncryption.java:
##########
@@ -285,7 +285,7 @@ public void testMetadataTamperproofing() throws IOException
{
assertThatThrownBy(() -> catalog.loadTable(tableIdent))
.hasMessageContaining(
String.format(
- "The current metadata file %s might have been modified. Hash
of metadata loaded from storage differs from HMS-stored metadata hash.",
Review Comment:
I moved the verifyMetadataHash method from `HMSTablePropertyHelper.java` to
`EncryptionUtil.java` , so I renamed the error message as well to make it
generic. I also fixed the assertion message accordingly
--
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]
