nsivabalan commented on code in PR #13085:
URL: https://github.com/apache/hudi/pull/13085#discussion_r2027766666
##########
hudi-client/hudi-client-common/src/test/java/org/apache/hudi/config/TestHoodieWriteConfig.java:
##########
@@ -103,7 +103,7 @@ public void testSupportedTableWriteVersions() {
Properties props = new Properties();
props.setProperty(HoodieWriteConfig.WRITE_TABLE_VERSION.key(),
String.valueOf(version.versionCode()));
assertThrows(IllegalArgumentException.class, () ->
HoodieWriteConfig.newBuilder()
- .withPath("/tmp").withProperties(props).build());
+
.withPath("/tmp").withProperties(props).build().getWriteVersion());
Review Comment:
We landed this patch w/o waiting for full CI run
https://github.com/apache/hudi/commit/c7c3532477a611d94b8bb53af592bde2f151d764
thinking it just a warn msg we are removing.
but apparently, one test is failing after this commit was merged.
Fixing the test in this patch to un block master.
--
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]
