[spark] branch branch-3.0 updated: [SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script Transform ROW FORMAT DELIMIT value should format value

2020-08-23 Thread viirya 
This is an automated email from the ASF dual-hosted git repository.

viirya pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
 new f5d5422  
[SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script 
Transform ROW FORMAT DELIMIT value should format value
f5d5422 is described below

commit f5d5422a4f87f69514d95f80f5f3db8246d61256
Author: angerszhu 
AuthorDate: Sun Aug 23 08:20:05 2020 -0700

[SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script 
Transform ROW FORMAT DELIMIT value should format value

### What changes were proposed in this pull request?
As mentioned in 
https://github.com/apache/spark/pull/29428#issuecomment-678735163 by viirya ,
fix bug in UT, since in script transformation no-serde mode, output of 
decimal is same in both hive-1.2/hive-2.3

### Why are the changes needed?
FIX UT

### Does this PR introduce _any_ user-facing change?
NO

### How was this patch tested?
EXISTED UT

Closes #29521 from AngersZh/SPARK-32608-3.0-FOLLOW-UP.

Authored-by: angerszhu 
Signed-off-by: Liang-Chi Hsieh 
---
 .../apache/spark/sql/hive/execution/ScriptTransformationSuite.scala   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
 
b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
index 15a932f..0d1fe20 100644
--- 
a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
+++ 
b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
@@ -299,7 +299,7 @@ class ScriptTransformationSuite extends SparkPlanTest with 
SQLTestUtils with Tes
   'a.cast("string"),
   'b.cast("string"),
   'c.cast("string"),
-  decimalToString('d),
+  'd.cast("string"),
   'e.cast("string")).collect())
 
   // input/output with different delimit and show result
@@ -322,7 +322,7 @@ class ScriptTransformationSuite extends SparkPlanTest with 
SQLTestUtils with Tes
 'a.cast("string"),
 'b.cast("string"),
 'c.cast("string"),
-decimalToString('d),
+'d.cast("string"),
 'e.cast("string"))).collect())
 }
   }


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

[spark] branch branch-3.0 updated: [SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script Transform ROW FORMAT DELIMIT value should format value

2020-08-23 Thread viirya 
This is an automated email from the ASF dual-hosted git repository.

viirya pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
 new f5d5422  
[SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script 
Transform ROW FORMAT DELIMIT value should format value
f5d5422 is described below

commit f5d5422a4f87f69514d95f80f5f3db8246d61256
Author: angerszhu 
AuthorDate: Sun Aug 23 08:20:05 2020 -0700

[SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script 
Transform ROW FORMAT DELIMIT value should format value

### What changes were proposed in this pull request?
As mentioned in 
https://github.com/apache/spark/pull/29428#issuecomment-678735163 by viirya ,
fix bug in UT, since in script transformation no-serde mode, output of 
decimal is same in both hive-1.2/hive-2.3

### Why are the changes needed?
FIX UT

### Does this PR introduce _any_ user-facing change?
NO

### How was this patch tested?
EXISTED UT

Closes #29521 from AngersZh/SPARK-32608-3.0-FOLLOW-UP.

Authored-by: angerszhu 
Signed-off-by: Liang-Chi Hsieh 
---
 .../apache/spark/sql/hive/execution/ScriptTransformationSuite.scala   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
 
b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
index 15a932f..0d1fe20 100644
--- 
a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
+++ 
b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
@@ -299,7 +299,7 @@ class ScriptTransformationSuite extends SparkPlanTest with 
SQLTestUtils with Tes
   'a.cast("string"),
   'b.cast("string"),
   'c.cast("string"),
-  decimalToString('d),
+  'd.cast("string"),
   'e.cast("string")).collect())
 
   // input/output with different delimit and show result
@@ -322,7 +322,7 @@ class ScriptTransformationSuite extends SparkPlanTest with 
SQLTestUtils with Tes
 'a.cast("string"),
 'b.cast("string"),
 'c.cast("string"),
-decimalToString('d),
+'d.cast("string"),
 'e.cast("string"))).collect())
 }
   }


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

[spark] branch branch-3.0 updated: [SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script Transform ROW FORMAT DELIMIT value should format value

2020-08-23 Thread viirya 
This is an automated email from the ASF dual-hosted git repository.

viirya pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
 new f5d5422  
[SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script 
Transform ROW FORMAT DELIMIT value should format value
f5d5422 is described below

commit f5d5422a4f87f69514d95f80f5f3db8246d61256
Author: angerszhu 
AuthorDate: Sun Aug 23 08:20:05 2020 -0700

[SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script 
Transform ROW FORMAT DELIMIT value should format value

### What changes were proposed in this pull request?
As mentioned in 
https://github.com/apache/spark/pull/29428#issuecomment-678735163 by viirya ,
fix bug in UT, since in script transformation no-serde mode, output of 
decimal is same in both hive-1.2/hive-2.3

### Why are the changes needed?
FIX UT

### Does this PR introduce _any_ user-facing change?
NO

### How was this patch tested?
EXISTED UT

Closes #29521 from AngersZh/SPARK-32608-3.0-FOLLOW-UP.

Authored-by: angerszhu 
Signed-off-by: Liang-Chi Hsieh 
---
 .../apache/spark/sql/hive/execution/ScriptTransformationSuite.scala   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
 
b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
index 15a932f..0d1fe20 100644
--- 
a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
+++ 
b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
@@ -299,7 +299,7 @@ class ScriptTransformationSuite extends SparkPlanTest with 
SQLTestUtils with Tes
   'a.cast("string"),
   'b.cast("string"),
   'c.cast("string"),
-  decimalToString('d),
+  'd.cast("string"),
   'e.cast("string")).collect())
 
   // input/output with different delimit and show result
@@ -322,7 +322,7 @@ class ScriptTransformationSuite extends SparkPlanTest with 
SQLTestUtils with Tes
 'a.cast("string"),
 'b.cast("string"),
 'c.cast("string"),
-decimalToString('d),
+'d.cast("string"),
 'e.cast("string"))).collect())
 }
   }


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

[spark] branch branch-3.0 updated: [SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script Transform ROW FORMAT DELIMIT value should format value

2020-08-23 Thread viirya 
This is an automated email from the ASF dual-hosted git repository.

viirya pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
 new f5d5422  
[SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script 
Transform ROW FORMAT DELIMIT value should format value
f5d5422 is described below

commit f5d5422a4f87f69514d95f80f5f3db8246d61256
Author: angerszhu 
AuthorDate: Sun Aug 23 08:20:05 2020 -0700

[SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script 
Transform ROW FORMAT DELIMIT value should format value

### What changes were proposed in this pull request?
As mentioned in 
https://github.com/apache/spark/pull/29428#issuecomment-678735163 by viirya ,
fix bug in UT, since in script transformation no-serde mode, output of 
decimal is same in both hive-1.2/hive-2.3

### Why are the changes needed?
FIX UT

### Does this PR introduce _any_ user-facing change?
NO

### How was this patch tested?
EXISTED UT

Closes #29521 from AngersZh/SPARK-32608-3.0-FOLLOW-UP.

Authored-by: angerszhu 
Signed-off-by: Liang-Chi Hsieh 
---
 .../apache/spark/sql/hive/execution/ScriptTransformationSuite.scala   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
 
b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
index 15a932f..0d1fe20 100644
--- 
a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
+++ 
b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
@@ -299,7 +299,7 @@ class ScriptTransformationSuite extends SparkPlanTest with 
SQLTestUtils with Tes
   'a.cast("string"),
   'b.cast("string"),
   'c.cast("string"),
-  decimalToString('d),
+  'd.cast("string"),
   'e.cast("string")).collect())
 
   // input/output with different delimit and show result
@@ -322,7 +322,7 @@ class ScriptTransformationSuite extends SparkPlanTest with 
SQLTestUtils with Tes
 'a.cast("string"),
 'b.cast("string"),
 'c.cast("string"),
-decimalToString('d),
+'d.cast("string"),
 'e.cast("string"))).collect())
 }
   }


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

[spark] branch branch-3.0 updated: [SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script Transform ROW FORMAT DELIMIT value should format value

2020-08-23 Thread viirya 
This is an automated email from the ASF dual-hosted git repository.

viirya pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
 new f5d5422  
[SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script 
Transform ROW FORMAT DELIMIT value should format value
f5d5422 is described below

commit f5d5422a4f87f69514d95f80f5f3db8246d61256
Author: angerszhu 
AuthorDate: Sun Aug 23 08:20:05 2020 -0700

[SPARK-32608][SQL][3.0][FOLLOW-UP][TEST-HADOOP2.7][TEST-HIVE1.2] Script 
Transform ROW FORMAT DELIMIT value should format value

### What changes were proposed in this pull request?
As mentioned in 
https://github.com/apache/spark/pull/29428#issuecomment-678735163 by viirya ,
fix bug in UT, since in script transformation no-serde mode, output of 
decimal is same in both hive-1.2/hive-2.3

### Why are the changes needed?
FIX UT

### Does this PR introduce _any_ user-facing change?
NO

### How was this patch tested?
EXISTED UT

Closes #29521 from AngersZh/SPARK-32608-3.0-FOLLOW-UP.

Authored-by: angerszhu 
Signed-off-by: Liang-Chi Hsieh 
---
 .../apache/spark/sql/hive/execution/ScriptTransformationSuite.scala   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
 
b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
index 15a932f..0d1fe20 100644
--- 
a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
+++ 
b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/ScriptTransformationSuite.scala
@@ -299,7 +299,7 @@ class ScriptTransformationSuite extends SparkPlanTest with 
SQLTestUtils with Tes
   'a.cast("string"),
   'b.cast("string"),
   'c.cast("string"),
-  decimalToString('d),
+  'd.cast("string"),
   'e.cast("string")).collect())
 
   // input/output with different delimit and show result
@@ -322,7 +322,7 @@ class ScriptTransformationSuite extends SparkPlanTest with 
SQLTestUtils with Tes
 'a.cast("string"),
 'b.cast("string"),
 'c.cast("string"),
-decimalToString('d),
+'d.cast("string"),
 'e.cast("string"))).collect())
 }
   }


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