cloud-fan commented on a change in pull request #29342:
URL: https://github.com/apache/spark/pull/29342#discussion_r470464327
##########
File path: core/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java
##########
@@ -601,6 +663,14 @@ public boolean isDefined() {
return isDefined;
}
+ /**
+ * Returns index for key.
Review comment:
I think in the `HashedRelation` we don't care too much about how the key
index is generated, and just require the index to be unique.
How about `Returns a unique index of the key.`
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
