(hudi) branch master updated: [MINOR] Fixing secondary index logging (#12397)

[codope]

This is an automated email from the ASF dual-hosted git repository.

codope pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hudi.git


The following commit(s) were added to refs/heads/master by this push:
     new 4e54b93c4d4  [MINOR] Fixing secondary index logging  (#12397)
4e54b93c4d4 is described below

commit 4e54b93c4d48d4dea11f0ac9554505c883559ddb
Author: Lokesh Jain <lj...@apache.org>
AuthorDate: Mon Dec 2 20:30:20 2024 +0530

     [MINOR] Fixing secondary index logging  (#12397)
    
    * Fixing secondary index logging
    
    * Fix test failure
    
    ---------
    
    Co-authored-by: sivabalan <n.siv...@gmail.com>
---
 .../org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java
 
b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java
index 86c26b2e015..a4e39998d1d 100644
--- 
a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java
+++ 
b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java
@@ -443,7 +443,9 @@ public abstract class HoodieBackedTableMetadataWriter<I> 
implements HoodieTableM
           case SECONDARY_INDEX:
             Set<String> secondaryIndexPartitionsToInit = 
getIndexPartitionsToInit(partitionType);
             if (secondaryIndexPartitionsToInit.size() != 1) {
-              LOG.warn("Skipping secondary index initialization as only one 
secondary index bootstrap at a time is supported for now. Provided: {}", 
secondaryIndexPartitionsToInit);
+              if (secondaryIndexPartitionsToInit.size() > 1) {
+                LOG.warn("Skipping secondary index initialization as only one 
secondary index bootstrap at a time is supported for now. Provided: {}", 
secondaryIndexPartitionsToInit);
+              }
               continue;
             }
             partitionName = secondaryIndexPartitionsToInit.iterator().next();