[GitHub] [hudi] YannByron commented on a change in pull request #3693: [HUDI-2456] support 'show partitions' sql

2021-09-28 Thread GitBox
YannByron commented on a change in pull request #3693: URL: https://github.com/apache/hudi/pull/3693#discussion_r717206741 ## File path: hudi-common/src/main/java/org/apache/hudi/common/util/PartitionPathEncodeUtils.java ## @@ -71,7 +73,7 @@ public static String

[GitHub] [hudi] YannByron commented on a change in pull request #3693: [HUDI-2456] support 'show partitions' sql

2021-09-27 Thread GitBox
YannByron commented on a change in pull request #3693: URL: https://github.com/apache/hudi/pull/3693#discussion_r717206741 ## File path: hudi-common/src/main/java/org/apache/hudi/common/util/PartitionPathEncodeUtils.java ## @@ -71,7 +73,7 @@ public static String

[GitHub] [hudi] YannByron commented on a change in pull request #3693: [HUDI-2456] support 'show partitions' sql

2021-09-26 Thread GitBox
YannByron commented on a change in pull request #3693: URL: https://github.com/apache/hudi/pull/3693#discussion_r716328913 ## File path: hudi-spark-datasource/hudi-spark/src/main/scala/org/apache/spark/sql/hudi/command/HoodieCommand.scala ## @@ -0,0 +1,47 @@ +/* + * Licensed

[GitHub] [hudi] YannByron commented on a change in pull request #3693: [HUDI-2456] support 'show partitions' sql

2021-09-26 Thread GitBox
YannByron commented on a change in pull request #3693: URL: https://github.com/apache/hudi/pull/3693#discussion_r716328861 ## File path: hudi-client/hudi-client-common/src/main/java/org/apache/hudi/keygen/KeyGenUtils.java ## @@ -40,7 +40,7 @@ protected static final String

[GitHub] [hudi] YannByron commented on a change in pull request #3693: [HUDI-2456] support 'show partitions' sql

2021-09-23 Thread GitBox
YannByron commented on a change in pull request #3693: URL: https://github.com/apache/hudi/pull/3693#discussion_r714882191 ## File path: hudi-common/src/main/java/org/apache/hudi/common/util/PartitionPathEncodeUtils.java ## @@ -25,6 +25,8 @@ */ public class

[GitHub] [hudi] YannByron commented on a change in pull request #3693: [HUDI-2456] support 'show partitions' sql

2021-09-23 Thread GitBox
YannByron commented on a change in pull request #3693: URL: https://github.com/apache/hudi/pull/3693#discussion_r714881110 ## File path: hudi-common/src/main/java/org/apache/hudi/common/util/PartitionPathEncodeUtils.java ## @@ -71,7 +73,7 @@ public static String

[GitHub] [hudi] YannByron commented on a change in pull request #3693: [HUDI-2456] support 'show partitions' sql

2021-09-23 Thread GitBox
YannByron commented on a change in pull request #3693: URL: https://github.com/apache/hudi/pull/3693#discussion_r714855194 ## File path: hudi-client/hudi-client-common/src/main/java/org/apache/hudi/keygen/KeyGenUtils.java ## @@ -40,7 +40,7 @@ protected static final String

[GitHub] [hudi] YannByron commented on a change in pull request #3693: [HUDI-2456] support 'show partitions' sql

2021-09-23 Thread GitBox
YannByron commented on a change in pull request #3693: URL: https://github.com/apache/hudi/pull/3693#discussion_r714855021 ## File path: hudi-spark-datasource/hudi-spark/src/main/scala/org/apache/spark/sql/hudi/command/HoodieCommand.scala ## @@ -0,0 +1,47 @@ +/* + * Licensed

[GitHub] [hudi] YannByron commented on a change in pull request #3693: [HUDI-2456] support 'show partitions' sql

2021-09-23 Thread GitBox
YannByron commented on a change in pull request #3693: URL: https://github.com/apache/hudi/pull/3693#discussion_r714849166 ## File path: hudi-spark-datasource/hudi-spark/src/test/scala/org/apache/spark/sql/hudi/HoodieSparkFunSuite.scala ## @@ -0,0 +1,199 @@ +/* + * Licensed