You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by vb...@apache.org on 2019/09/13 19:41:57 UTC

[incubator-hudi] branch master updated: [HUDI-244] : Hive sync should escape partition field name

This is an automated email from the ASF dual-hosted git repository.

vbalaji pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-hudi.git


The following commit(s) were added to refs/heads/master by this push:
     new b1446be  [HUDI-244] : Hive sync should escape partition field name
b1446be is described below

commit b1446be2b4d50e60b54146c66a6e6412d41b3a17
Author: Vinoth Chandar <vi...@apache.org>
AuthorDate: Wed Sep 11 15:47:22 2019 -0700

    [HUDI-244] : Hive sync should escape partition field name
    
      - now supports field names beginning with '_' for e.g
---
 hudi-hive/src/main/java/org/apache/hudi/hive/util/SchemaUtil.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hudi-hive/src/main/java/org/apache/hudi/hive/util/SchemaUtil.java b/hudi-hive/src/main/java/org/apache/hudi/hive/util/SchemaUtil.java
index 6cd7747..289f2f7 100644
--- a/hudi-hive/src/main/java/org/apache/hudi/hive/util/SchemaUtil.java
+++ b/hudi-hive/src/main/java/org/apache/hudi/hive/util/SchemaUtil.java
@@ -410,7 +410,7 @@ public class SchemaUtil {
     List<String> partitionFields = new ArrayList<>();
     for (String partitionKey : config.partitionFields) {
       String partitionKeyWithTicks = tickSurround(partitionKey);
-      partitionFields.add(new StringBuilder().append(partitionKey).append(" ")
+      partitionFields.add(new StringBuilder().append(partitionKeyWithTicks).append(" ")
           .append(getPartitionKeyType(hiveSchema, partitionKeyWithTicks)).toString());
     }