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

    https://github.com/apache/spark/pull/22349#discussion_r216120943
  
    --- Diff: python/pyspark/ml/image.py ---
    @@ -30,6 +30,7 @@
     from pyspark import SparkContext
     from pyspark.sql.types import Row, _create_row, _parse_datatype_json_string
     from pyspark.sql import DataFrame, SparkSession
    +import warnings
    --- End diff --
    
    Technically builtin package should be ordered above per PEP 8. I wonder why 
this is not caught.
    
    ```
    import sys
    import warnings
     
    import numpy as np
    
    from pyspark import SparkContext
    from pyspark.sql.types import Row, _create_row, _parse_datatype_json_string
    from pyspark.sql import DataFrame, SparkSession
    ```


---

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

Reply via email to