FYI once this pull request goes in, scalastyle will be run against all test code as well. Previously we didn't run the style checker on test code.
https://github.com/apache/spark/pull/5486 We might see the master build breaking tomorrow if a PR has style violations in test code and is tested by Jenkins before the above PR is merged.