nsivabalan commented on a change in pull request #4693:
URL: https://github.com/apache/hudi/pull/4693#discussion_r835779230



##########
File path: 
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java
##########
@@ -392,6 +398,12 @@ public void initTableMetadata() {
     }
 
     if (!exists) {
+      if (metadataWriteConfig.isMetadataAsyncIndex()) {

Review comment:
       guess we missed one flow here. 
   lets say someone brought down every writer. And starts HoodieIndexer for the 
first time by enabling all partitions with the intention that everything will 
get built out. I see we come to this code path and eventually call 
scheduleIndex in L404, including FILES partition. Guess the plan is to always 
follow synchronous code path for FILES partition. 
   




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