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

    https://github.com/apache/spark/pull/22963#discussion_r231460594
  
    --- Diff: dev/lint-python ---
    @@ -26,9 +26,13 @@ 
PYCODESTYLE_REPORT_PATH="$SPARK_ROOT_DIR/dev/pycodestyle-report.txt"
     PYDOCSTYLE_REPORT_PATH="$SPARK_ROOT_DIR/dev/pydocstyle-report.txt"
     PYLINT_REPORT_PATH="$SPARK_ROOT_DIR/dev/pylint-report.txt"
     PYLINT_INSTALL_INFO="$SPARK_ROOT_DIR/dev/pylint-info.txt"
    +
     PYDOCSTYLEBUILD="pydocstyle"
    -EXPECTED_PYDOCSTYLEVERSION="3.0.0"
    -PYDOCSTYLEVERSION=$(python -c 'import pkg_resources; 
print(pkg_resources.get_distribution("pydocstyle").version)' 2> /dev/null)
    +MINIMUM_PYDOCSTYLEVERSION="3.0.0"
    +
    +FLAKE8BUILD="flake8"
    +MINIMUM_FLAKE8="3.5.0"
    --- End diff --
    
    @shaneknapp and @cloud-fan, it seems flake8 does work but indeed the 
problem was compatibility with pycodestyle within flake8 itself.
    
    Our script uses pycodestyle 2.4.0 but it's manually downloaded. So, 
Jenkins's pycodestyle might be different or old and this might be the actual 
root cause.
    
    I locally tested with latest pycodestyle and flake 3.6.0 and it worked.


---

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

Reply via email to