Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/1744#discussion_r15782822
--- Diff: python/pyspark/__init__.py ---
@@ -53,7 +53,8 @@
# mllib that depend on top level pyspark packages, which transitively
depend on python's random.
# Since Python's import logic looks for modules in the current package
first, we eliminate
# mllib.random as a candidate for C{import random} by removing the first
search path, the script's
-# location, in order to force the loader to look in Python's top-level
modules for C{random}.
+# location, in order to force the loader to look in Python's top-level
--- End diff --
In general, it looks like a bunch of these changes are related to line
length, but `autopep8` seems to have done it fairly haphazardly. I think
100-character lines are probably fine, since that seems to be the standard that
we've been using.
---
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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
