Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/19160#discussion_r139796177 --- Diff: core/src/main/scala/org/apache/spark/network/netty/NettyBlockTransferService.scala --- @@ -18,11 +18,14 @@ package org.apache.spark.network.netty import java.nio.ByteBuffer +import java.util --- End diff -- if you are trying to avoid confusion w/ scala's hashmaps, I think our convention is to rename w/ "J" prefix ``` import java.util.{HashMap => JHashMap} ```
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org