BryanCutler commented on a change in pull request #24177: [SPARK-27240][PYTHON]
Use pandas DataFrame for struct type argument in Scalar Pandas UDF.
URL: https://github.com/apache/spark/pull/24177#discussion_r268784079
##########
File path: python/pyspark/serializers.py
##########
@@ -378,6 +379,29 @@ class
ArrowStreamPandasUDFSerializer(ArrowStreamPandasSerializer):
Serializer used by Python worker to evaluate Pandas UDFs
"""
+ def __init__(self, timezone, safecheck, assign_cols_by_name,
df_for_struct=False):
+ super(ArrowStreamPandasUDFSerializer, self) \
+ .__init__(timezone, safecheck, assign_cols_by_name)
+ self._df_for_struct = df_for_struct
+
+ def arrow_to_pandas(self, arrow_column, data_type):
+ from pyspark.sql.types import StructType, \
+ _arrow_column_to_pandas, _check_dataframe_localize_timestamps
+
+ if self._df_for_struct and type(data_type) == StructType:
Review comment:
I was wondering if `data_type` has a nested struct, then is an error raised
before it gets here? That could be addressed as a followup, I'm not sure if
there is a test for it, but I'll check.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
