Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20507
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20507#discussion_r166168248
--- Diff: python/pyspark/serializers.py ---
@@ -230,6 +230,9 @@ def create_array(s, t):
s = _check_series_convert_timestamps_internal(s.filln
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20507#discussion_r166018470
--- Diff: python/pyspark/serializers.py ---
@@ -230,6 +230,9 @@ def create_array(s, t):
s = _check_series_convert_timestamps_internal(s.
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20507#discussion_r165980572
--- Diff: python/pyspark/sql/tests.py ---
@@ -3920,6 +3920,14 @@ def test_vectorized_udf_null_string(self):
res = df.select(str_f(col('str')))
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20507#discussion_r165980594
--- Diff: python/pyspark/sql/tests.py ---
@@ -3920,6 +3920,14 @@ def test_vectorized_udf_null_string(self):
res = df.select(str_f(col('str')))
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20507#discussion_r165972212
--- Diff: python/pyspark/sql/tests.py ---
@@ -3920,6 +3920,14 @@ def test_vectorized_udf_null_string(self):
res = df.select(str_f(col('str')
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20507#discussion_r165968902
--- Diff: python/pyspark/sql/tests.py ---
@@ -3920,6 +3920,14 @@ def test_vectorized_udf_null_string(self):
res = df.select(str_f(col('str')
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/20507
[SPARK-23334][SQL][PYTHON] Fix pandas_udf with return type StringType() to
handle str type properly in Python 2.
## What changes were proposed in this pull request?
In Python 2, when `panda