Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21495#discussion_r196878408
--- Diff:
repl/scala-2.11/src/main/scala/org/apache/spark/repl/SparkILoopInterpreter.scala
---
@@ -21,8 +21,22 @@ import scala.collection.mutable
import scala.tools.nsc.Settings
import scala.tools.nsc.interpreter._
-class SparkILoopInterpreter(settings: Settings, out: JPrintWriter) extends
IMain(settings, out) {
- self =>
+class SparkILoopInterpreter(settings: Settings, out: JPrintWriter,
initializeSpark: () => Unit)
+ extends IMain(settings, out) { self =>
--- End diff --
Guys, parameters are listed in 4 spaces and other keywords after that are
lined up with 2 spaces, which is written in
https://github.com/databricks/scala-style-guide#spacing-and-indentation
In case of two lines, it's not explicitly written but wouldn't we better
stick to the example as possible as we can?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
