Github user JoshRosen commented on the pull request:

    https://github.com/apache/spark/pull/6707#issuecomment-184022738
  
    I know it's about half a year too late to be complaining about this, but 
the formatting and control flow in this patch's tests make things really hard 
to debug and there's a lot of code formatting issues here. Next time, I think 
we should push for stronger test code quality on patches like this one, since 
this one is a giant mess and is very unpleasant to try to understand / edit.


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