[SPARK-3854] Scala style: require spaces before `{`

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.0.0
• Component/s: Project Infra
• Target Version/s: 2.0.0
• Assignee: Dongjoon Hyun
• Reporter: Josh Rosen
• Created: 08/Oct/14 21:04
• Updated: 07/Feb/20 17:23
• Resolved: 10/Mar/16 23:57

Description

We should require spaces before opening curly braces. This isn't in the style guide, but it probably should be:

// Correct:
if (true) {
  println("Wow!")
}

// Incorrect:
if (true){
   println("Wow!")
}

See https://github.com/apache/spark/pull/1658#discussion-diff-18611791 for an example "in the wild."

git grep "){" shows only a few occurrences of this style.

Issue Links

• links to: [Github] Pull Request #11637 (dongjoon-hyun)