nsivabalan commented on code in PR #13346:
URL: https://github.com/apache/hudi/pull/13346#discussion_r2109772357


##########
hudi-client/hudi-spark-client/src/test/java/org/apache/hudi/client/functional/TestHoodieBackedTableMetadata.java:
##########
@@ -411,6 +411,61 @@ public void 
testRepeatedCleanActionsWithMetadataTableEnabled(final HoodieTableTy
     assertEquals(tableVersion, finalTableVersion.versionCode());
   }
 
+  @ParameterizedTest
+  @CsvSource({"COPY_ON_WRITE,6", "COPY_ON_WRITE,8", "MERGE_ON_READ,6", 
"MERGE_ON_READ,8"})
+  void testDeletePartitionKeyOnCleanPartition(final HoodieTableType tableType, 
int tableVersion) throws Exception {

Review Comment:
   we dont' need to make this run for 4 combinations. lets not try to add more 
parametrized test if there is no need. 
   this is just testing regular updates to FILES partition in MDT. 
   just 1 version would suffice. 
   



-- 
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]

Reply via email to