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

    https://github.com/apache/spark/pull/19646#discussion_r149863279
  
    --- Diff: python/pyspark/sql/tests.py ---
    @@ -2592,6 +2592,21 @@ def test_create_dataframe_from_array_of_long(self):
             df = self.spark.createDataFrame(data)
             self.assertEqual(df.first(), Row(longarray=[-9223372036854775808, 
0, 9223372036854775807]))
     
    +    @unittest.skipIf(not _have_pandas, "Pandas not installed")
    --- End diff --
    
    Thanks @dongjoon-hyun for tracking this down.  It looks like sql/tests.py 
for branch-2.2 is just missing the following
    
    ```
    
    _have_pandas = False
    try:
        import pandas
        _have_pandas = True
    except:
        # No Pandas, but that's okay, we'll skip those tests
        pass
    ```
    This was probably adding from an earlier PR in master and wasn't included 
when this was cherry-picked.


---

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

Reply via email to