Github user holdenk commented on the issue:

    https://github.com/apache/spark/pull/14830
  
    For what its worth pep8 says:
    
    > The preferred way of wrapping long lines is by using Python's implied 
line continuation inside parentheses, brackets and braces. Long lines can be 
broken over multiple lines by wrapping expressions in parentheses. These should 
be used in preference to using a backslash for line continuation.
    
    So this sounds like keeping in line with the general more pep8ification of 
the code - but I am a little concerned about just how many files this touches 
now that it isn't just an autogenerated change*, but I'll try and set aside 
some time this week to review it (I'm currently ~13 hours off my regular 
timezone so my review times may be a little erratic).


---
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]

Reply via email to