[GitHub] [hudi] wangxianghu commented on a change in pull request #2449: [HUDI-1528] hudi-sync-tools supports synchronization to remote hive

[GitBox <gi...@apache.org>]

wangxianghu commented on a change in pull request #2449:
URL: https://github.com/apache/hudi/pull/2449#discussion_r557806942



##########
File path: hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HiveSyncConfig.java
##########
@@ -49,6 +49,9 @@
   @Parameter(names = {"--jdbc-url"}, description = "Hive jdbc connect url", required = true)
   public String jdbcUrl;
 
+  @Parameter(names = {"--metastore-uri"}, description = "Hive metastore uri)")
+  public String hiveMetaStoreUri;
+
   @Parameter(names = {"--base-path"}, description = "Basepath of hoodie table to sync", required = true)

Review comment:
       pleasse improve the rescription that this paramter is required when sync data to remote hive




----------------------------------------------------------------
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:
users@infra.apache.org