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

    https://github.com/apache/spark/pull/20487#discussion_r165883676
  
    --- Diff: python/pyspark/sql/dataframe.py ---
    @@ -1923,6 +1923,9 @@ def toPandas(self):
             0    2  Alice
             1    5    Bob
             """
    +        from pyspark.sql.utils import require_minimum_pandas_version
    --- End diff --
    
    We should also add `require_minimum_pandas_version()` to line 649 in 
session.py?
    
https://github.com/apache/spark/blob/a0e4b166f71f9bb5f3e5af7843a03c11658892fd/python/pyspark/sql/session.py#L643-L653


---

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

Reply via email to