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

morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new b28f31f98d [fix](meta) fix show create table result of hive table 
(#17677)
b28f31f98d is described below

commit b28f31f98dce2957751a6ff2462795a54727c9c2
Author: Yulei-Yang <yulei.yang0...@gmail.com>
AuthorDate: Tue Mar 14 22:50:35 2023 +0800

    [fix](meta) fix show create table result of hive table (#17677)
    
    make it usable in hive.
    
    current issue: type of partition column are wrapped by ``, it's not illegal 
in hive. One problem case:
    
    CREATE TABLE t3p_parquet(
    id int,
    name string)
    PARTITIONED BY (
    dt int)
    ROW FORMAT SERDE
    'org.apache.hadoop.hive.ql.io.parquet.serde.ParquetHiveSerDe'
    STORED AS INPUTFORMAT
    'org.apache.hadoop.hive.ql.io.parquet.MapredParquetInputFormat'
    OUTPUTFORMAT
    'org.apache.hadoop.hive.ql.io.parquet.MapredParquetOutputFormat'
    LOCATION
    'hdfs://path/to/t3p_parquet'
    TBLPROPERTIES (
    'transient_lastDdlTime'='1671700883')
---
 .../main/java/org/apache/doris/catalog/HiveMetaStoreClientHelper.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/catalog/HiveMetaStoreClientHelper.java
 
b/fe/fe-core/src/main/java/org/apache/doris/catalog/HiveMetaStoreClientHelper.java
index 53b492082f..e4f2ad993b 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/catalog/HiveMetaStoreClientHelper.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/catalog/HiveMetaStoreClientHelper.java
@@ -824,7 +824,7 @@ public class HiveMetaStoreClientHelper {
                 output.append("PARTITIONED BY (\n")
                         .append(remoteTable.getPartitionKeys().stream().map(
                                         partition ->
-                                                String.format(" `%s` `%s`", 
partition.getName(), partition.getType()))
+                                                String.format(" `%s` %s", 
partition.getName(), partition.getType()))
                                 .collect(Collectors.joining(",\n")))
                         .append(")\n");
             }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to