[GitHub] [spark] TisonKun commented on a change in pull request #24366: [MINOR] Fix import in NettyStreamManager.scala

[GitBox <gi...@apache.org>]

TisonKun commented on a change in pull request #24366: [MINOR] Fix import in NettyStreamManager.scala
URL: https://github.com/apache/spark/pull/24366#discussion_r275119749
 
 

 ##########
 File path: core/src/main/scala/org/apache/spark/rpc/netty/NettyStreamManager.scala
 ##########
 @@ -19,6 +19,7 @@ package org.apache.spark.rpc.netty
 import java.io.File
 import java.util.concurrent.ConcurrentHashMap
 
+import org.apache.spark.SparkContext
 
 Review comment:
   We refer to `[[SparkContext.addFile]]` below in scala doc. This import fix the link, otherwise scala doc cannot find where `[[SparkContext.addFile]]` should be linked to.
   
   As the other comment of yours, adding this doesn't affect production code logic though.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org