Github user BryanCutler commented on the issue:

    https://github.com/apache/spark/pull/21397
  
    This seems like it should be good to me.  It's a little bit different than 
the ArrowConverters that also have a listener, because they are iterators and 
the cleanup can't be put in a finally.  I would like for @ueshin to take a look 
though.
    
    Also, I don't think we should include the unit test if it doesn't create 
the issue every time.


---

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

Reply via email to