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

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


The following commit(s) were added to refs/heads/master by this push:
     new ac60b977fe3 HIVE-27439: Support space in Decimal(precision, scale) 
format (#4417). (leesf, reviewed by Ayush Saxena, Butao Zhang)
ac60b977fe3 is described below

commit ac60b977fe3db6af268a711853a2e42e43d5d445
Author: leesf <[email protected]>
AuthorDate: Fri Jun 23 20:50:12 2023 +0800

    HIVE-27439: Support space in Decimal(precision, scale) format (#4417). 
(leesf, reviewed by Ayush Saxena, Butao Zhang)
---
 .../java/org/apache/hadoop/hive/serde2/typeinfo/TypeInfoUtils.java | 2 +-
 .../org/apache/hadoop/hive/serde2/typeinfo/TestTypeInfoUtils.java  | 7 ++++++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git 
a/serde/src/java/org/apache/hadoop/hive/serde2/typeinfo/TypeInfoUtils.java 
b/serde/src/java/org/apache/hadoop/hive/serde2/typeinfo/TypeInfoUtils.java
index b23e3cfc445..4788558d4b8 100644
--- a/serde/src/java/org/apache/hadoop/hive/serde2/typeinfo/TypeInfoUtils.java
+++ b/serde/src/java/org/apache/hadoop/hive/serde2/typeinfo/TypeInfoUtils.java
@@ -334,7 +334,7 @@ public final class TypeInfoUtils {
             || !isTypeChar(typeInfoString.charAt(end))) {
           Token t = new Token();
           t.position = begin;
-          t.text = typeInfoString.substring(begin, end);
+          t.text = typeInfoString.substring(begin, end).trim();
           t.isType = isTypeChar(typeInfoString.charAt(begin));
           tokens.add(t);
           begin = end;
diff --git 
a/serde/src/test/org/apache/hadoop/hive/serde2/typeinfo/TestTypeInfoUtils.java 
b/serde/src/test/org/apache/hadoop/hive/serde2/typeinfo/TestTypeInfoUtils.java
index db25db21957..a29f5d1fe66 100644
--- 
a/serde/src/test/org/apache/hadoop/hive/serde2/typeinfo/TestTypeInfoUtils.java
+++ 
b/serde/src/test/org/apache/hadoop/hive/serde2/typeinfo/TestTypeInfoUtils.java
@@ -47,7 +47,12 @@ public class TestTypeInfoUtils {
         "string",
         "varchar(10)",
         "char(15)",
-        "array<int>"
+        "array<int>",
+        "decimal(10,2)",
+        "decimal(10, 2)",
+        "decimal(10, 2 )",
+        "decimal( 10, 2 )",
+        "struct<user id:int,user group: int>"
     };
 
     String[] invalidTypeStrings = {

Reply via email to