[GitHub] spark pull request: follow pep8 None should be compared using is o...

2014-07-15 Thread giwa
GitHub user giwa opened a pull request: https://github.com/apache/spark/pull/1422 follow pep8 None should be compared using is or is not http://legacy.python.org/dev/peps/pep-0008/ ## Programming Recommendations - Comparisons to singletons like None should always

[GitHub] spark pull request: follow pep8 None should be compared using is o...

2014-07-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/1422#issuecomment-49098978 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: follow pep8 None should be compared using is o...

2014-07-15 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/1422#issuecomment-49100610 Jenkins, test this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: follow pep8 None should be compared using is o...

2014-07-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1422#issuecomment-49101045 QA tests have started for PR 1422. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16695/consoleFull ---

[GitHub] spark pull request: follow pep8 None should be compared using is o...

2014-07-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1422#issuecomment-49108961 QA results for PR 1422:br- This patch PASSES unit tests.br- This patch merges cleanlybr- This patch adds no public classesbrbrFor more information see test