[GitHub] [hudi] lhjzmn commented on a change in pull request #2532: [HUDI-1534]HiveSyncTool-It is not necessary to use JDBC and MetaStoreClient at the same time

2021-02-22 Thread GitBox
lhjzmn commented on a change in pull request #2532: URL: https://github.com/apache/hudi/pull/2532#discussion_r580796528 ## File path: hudi-sync/hudi-hive-sync/src/test/java/org/apache/hudi/hive/TestHiveSyncTool.java ## @@ -87,30 +89,30 @@ public void testSchemaConvertArray()

[GitHub] [hudi] lhjzmn commented on a change in pull request #2532: [HUDI-1534]HiveSyncTool-It is not necessary to use JDBC and MetaStoreClient at the same time

2021-02-21 Thread GitBox
lhjzmn commented on a change in pull request #2532: URL: https://github.com/apache/hudi/pull/2532#discussion_r579925386 ## File path: hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HoodieHiveClient.java ## @@ -343,116 +304,48 @@ public boolean

[GitHub] [hudi] lhjzmn commented on a change in pull request #2532: [HUDI-1534]HiveSyncTool-It is not necessary to use JDBC and MetaStoreClient at the same time

2021-02-21 Thread GitBox
lhjzmn commented on a change in pull request #2532: URL: https://github.com/apache/hudi/pull/2532#discussion_r579912713 ## File path: hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HoodieHiveClient.java ## @@ -64,32 +56,23 @@ public class HoodieHiveClient

[GitHub] [hudi] lhjzmn commented on a change in pull request #2532: [HUDI-1534]HiveSyncTool-It is not necessary to use JDBC and MetaStoreClient at the same time

2021-02-19 Thread GitBox
lhjzmn commented on a change in pull request #2532: URL: https://github.com/apache/hudi/pull/2532#discussion_r579582711 ## File path: hudi-sync/hudi-dla-sync/src/main/java/org/apache/hudi/dla/util/DLASchemaUtil.java ## @@ -0,0 +1,432 @@ +/* + * Licensed to the Apache Software

[GitHub] [hudi] lhjzmn commented on a change in pull request #2532: [HUDI-1534]HiveSyncTool-It is not necessary to use JDBC and MetaStoreClient at the same time

2021-02-19 Thread GitBox
lhjzmn commented on a change in pull request #2532: URL: https://github.com/apache/hudi/pull/2532#discussion_r579581652 ## File path: hudi-sync/hudi-dla-sync/src/main/java/org/apache/hudi/dla/util/DLASchemaUtil.java ## @@ -0,0 +1,432 @@ +/* + * Licensed to the Apache Software