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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
