Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18659#discussion_r139289505
  
    --- Diff: python/pyspark/serializers.py ---
    @@ -199,6 +211,33 @@ def __repr__(self):
             return "ArrowSerializer"
     
     
    +class ArrowPandasSerializer(ArrowSerializer):
    +
    +    def __init__(self):
    +        super(ArrowPandasSerializer, self).__init__()
    +
    +    def dumps(self, series):
    +        """
    +        Make an ArrowRecordBatch from a Pandas Series and serialize
    +        """
    +        import pyarrow as pa
    --- End diff --
    
    Ah, I see. Thanks for explaining it. Sure, I am okay with leaving it as is 
here.
    I think we should catch and throw it with better messages in all cases 
(probably in the entry points) but probably let's talk about this later in 
another place.


---

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

Reply via email to