diff options
| author | Sean Owen <sowen@cloudera.com> | 2014-09-02 10:30:26 -0700 |
|---|---|---|
| committer | Josh Rosen <joshrosen@apache.org> | 2014-09-02 10:30:26 -0700 |
| commit | 32ec0a8cd4020f1e8ef2997de310823b566ab2de (patch) | |
| tree | 638c36b7b66647e125b0f79c80b73b0282dddebc /dev | |
| parent | 0f16b23cd17002fac05f3ecc58899be1b1121b82 (diff) | |
| download | spark-32ec0a8cd4020f1e8ef2997de310823b566ab2de.tar.gz spark-32ec0a8cd4020f1e8ef2997de310823b566ab2de.tar.bz2 spark-32ec0a8cd4020f1e8ef2997de310823b566ab2de.zip | |
SPARK-3331 [BUILD] PEP8 tests fail because they check unzipped py4j code
PEP8 tests run on files under "./python", but unzipped py4j code is found at "./python/build/py4j". Py4J code fails style checks and can fail ./dev/run-tests if this code is present locally.
Author: Sean Owen <sowen@cloudera.com>
Closes #2222 from srowen/SPARK-3331 and squashes the following commits:
34711ec [Sean Owen] Restrict lint check to pyspark/, since the local directory can contain unzipped py4j code in build/py4j
Diffstat (limited to 'dev')
| -rwxr-xr-x | dev/lint-python | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/dev/lint-python b/dev/lint-python index 4efddad839..a1e890faa8 100755 --- a/dev/lint-python +++ b/dev/lint-python @@ -44,7 +44,7 @@ fi #+ first, but we do so so that the check status can #+ be output before the report, like with the #+ scalastyle and RAT checks. -python $PEP8_SCRIPT_PATH ./python > "$PEP8_REPORT_PATH" +python $PEP8_SCRIPT_PATH ./python/pyspark > "$PEP8_REPORT_PATH" pep8_status=${PIPESTATUS[0]} #$? if [ $pep8_status -ne 0 ]; then |