prashantwason commented on code in PR #8684:
URL: https://github.com/apache/hudi/pull/8684#discussion_r1192166826


##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java:
##########
@@ -873,17 +908,7 @@ public void buildMetadataPartitions(HoodieEngineContext 
engineContext, List<Hood
     indexPartitionInfos.forEach(indexPartitionInfo -> {
       String relativePartitionPath = 
indexPartitionInfo.getMetadataPartitionPath();
       LOG.info(String.format("Creating a new metadata index for partition '%s' 
under path %s upto instant %s",
-          relativePartitionPath, metadataWriteConfig.getBasePath(), 
indexUptoInstantTime));
-      try {
-        // file group should have already been initialized while scheduling 
index for this partition
-        if (!dataMetaClient.getFs().exists(new 
Path(metadataWriteConfig.getBasePath(), relativePartitionPath))) {

Review Comment:
   HoodieBackedTableMetadataWriter::initializeFromFilesystem
    There is a call to initializeFileGroups()
   
   Within initializeFileGroups() existing fileSlices should be deleted (I filed 
a separate PR for this - the one where the fileSlices are created in parallel).
   



-- 
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: commits-unsubscr...@hudi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to