satishkotha commented on a change in pull request #2918:
URL: https://github.com/apache/hudi/pull/2918#discussion_r632102292



##########
File path: 
hudi-client/hudi-spark-client/src/test/java/org/apache/hudi/client/TestHoodieClientOnCopyOnWriteStorage.java
##########
@@ -1114,6 +1114,16 @@ public void testClusteringWithSortColumns() throws 
Exception {
         
.withClusteringTargetPartitions(0).withInlineClusteringNumCommits(1).build();
     testClustering(clusteringConfig);
   }
+  
+  @Test
+  public void testClusteringWithOneFilePerGroup() throws Exception {
+    HoodieClusteringConfig clusteringConfig = 
HoodieClusteringConfig.newBuilder().withClusteringMaxNumGroups(10)
+        .withClusteringMaxBytesInGroup(10) // set small number so each file is 
considered as separate clustering group
+        
.withClusteringExecutionStrategyClass("org.apache.hudi.ClusteringIdentityTestExecutionStrategy")

Review comment:
       This is just a unit test. Will provide another schedule clustering 
strategy as part of another PR to limit number of files per group.




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to