Hyukjin Kwon created SPARK-33749:
------------------------------------
Summary: Exclude cloudpickle and conf.py more permissively in
pycodestyle and flake8
Key: SPARK-33749
URL: https://issues.apache.org/jira/browse/SPARK-33749
Project: Spark
Issue Type: Bug
Components: Build
Affects Versions: 3.1.0
Reporter: Hyukjin Kwon
Once you build and ran K8S tests, Python lint fails as below:
{code:java}
$ ./dev/lint-python
starting python compilation test...
python compilation succeeded.
downloading pycodestyle from
https://raw.githubusercontent.com/PyCQA/pycodestyle/2.6.0/pycodestyle.py...
starting pycodestyle test...
pycodestyle checks failed:
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/pyspark/cloudpickle/cloudpickle.py:15:101:
E501 line too long (105 > 100 characters)
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:60:101:
E501 line too long (124 > 100 characters)
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:76:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:96:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:100:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:102:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:110:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:113:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:117:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:121:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:127:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:130:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:147:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:150:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:154:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:157:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:166:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:180:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:184:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:188:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:191:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:195:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:204:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:207:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:210:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:213:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:218:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:221:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:230:1:
E122 continuation line missing indentation or outdented
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:231:1:
E122 continuation line missing indentation or outdented
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:231:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:233:1:
E122 continuation line missing indentation or outdented
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:234:1:
E122 continuation line missing indentation or outdented
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:234:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:236:1:
E122 continuation line missing indentation or outdented
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:237:1:
E122 continuation line missing indentation or outdented
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:237:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:244:3:
E121 continuation line under-indented for hanging indent
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:250:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:254:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:257:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:260:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:263:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:266:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:279:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:288:3:
E121 continuation line under-indented for hanging indent
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:294:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:297:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:300:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:303:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:315:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:321:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:325:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:328:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:332:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:335:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:338:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:341:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:345:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:349:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:355:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:358:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:361:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:364:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:367:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:370:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:373:1:
E265 block comment should start with '# '
./resource-managers/kubernetes/integration-tests/target/spark-dist-unpacked/python/docs/source/conf.py:374:1:
E302 expected 2 blank lines, found 1
{code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
