danny0405 commented on code in PR #11081:
URL: https://github.com/apache/hudi/pull/11081#discussion_r1577097199


##########
hudi-common/src/main/java/org/apache/hudi/common/table/HoodieTableMetaClient.java:
##########
@@ -584,6 +585,39 @@ public static HoodieTableMetaClient 
initTableAndGetMetaClient(Configuration hado
     return metaClient;
   }
 
+  /**
+   * @param conf     file system configuration.
+   * @param basePath base path of the Hudi table.
+   * @return a new {@link HoodieTableMetaClient} instance.
+   */
+  public static HoodieTableMetaClient build(Configuration conf,
+                                            String basePath) {

Review Comment:
   Don't think there is necessity to add three new builder methods, the 
original builder is more flexible to extend and there is no much gains to 
switch these new methods which also introduce bunden for maintainance.



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