Repository: spark Updated Branches: refs/heads/master 8f74aa639 -> 60851bc7b
[SPARK-11110][BUILD] Remove transient annotation for parameters. `transient` annotations on class parameters (not case class parameters or vals) causes compilation errors during compilation with Scala 2.11. I understand that transient *parameters* make no sense, however I don't quite understand why the 2.10 compiler accepted them. Note: in case it is preferred to keep the annotations in case someone would in the future want to redefine them as vals, it would also be possible to just add `val` after the annotation, e.g. `class Foo(transient x: Int)` becomes `class Foo(transient private val x: Int)`. I chose to remove the annotation as it also reduces needles clutter, however please feel free to tell me if you prefer the second option and I'll update the PR Author: Jakob Odersky <[email protected]> Closes #9126 from jodersky/sbt-scala-2.11. Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/60851bc7 Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/60851bc7 Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/60851bc7 Branch: refs/heads/master Commit: 60851bc7bf1fe702cde1d1548efd8016411e1682 Parents: 8f74aa6 Author: Jakob Odersky <[email protected]> Authored: Tue Oct 20 08:54:34 2015 +0100 Committer: Sean Owen <[email protected]> Committed: Tue Oct 20 08:54:34 2015 +0100 ---------------------------------------------------------------------- .../main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala | 2 +- .../src/main/scala/org/apache/spark/sql/hive/HiveContext.scala | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/spark/blob/60851bc7/core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala ---------------------------------------------------------------------- diff --git a/core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala b/core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala index a2b28c5..e01cf1a 100644 --- a/core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala +++ b/core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala @@ -308,7 +308,7 @@ private[netty] class NettyRpcEnvFactory extends RpcEnvFactory with Logging { } } -private[netty] class NettyRpcEndpointRef(@transient conf: SparkConf) +private[netty] class NettyRpcEndpointRef(@transient private val conf: SparkConf) extends RpcEndpointRef(conf) with Serializable with Logging { @transient @volatile private var nettyEnv: NettyRpcEnv = _ http://git-wip-us.apache.org/repos/asf/spark/blob/60851bc7/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala ---------------------------------------------------------------------- diff --git a/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala b/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala index 4d8a3f7..38c195b 100644 --- a/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala +++ b/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala @@ -89,9 +89,9 @@ private[hive] case class CurrentDatabase(ctx: HiveContext) class HiveContext private[hive]( sc: SparkContext, cacheManager: CacheManager, - @transient listener: SQLListener, - @transient execHive: ClientWrapper, - @transient metaHive: ClientInterface, + listener: SQLListener, + @transient private val execHive: ClientWrapper, + @transient private val metaHive: ClientInterface, isRootContext: Boolean) extends SQLContext(sc, cacheManager, listener, isRootContext) with Logging { self => --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
