kbendick commented on a change in pull request #748:
URL: https://github.com/apache/orc/pull/748#discussion_r676347645
##########
File path: java/core/src/java/org/apache/orc/impl/StringHashTableDictionary.java
##########
@@ -173,10 +170,18 @@ public int size() {
/**
* Compute the hash value and find the corresponding index.
- *
*/
int getIndex(Text text) {
- return Math.floorMod(text.hashCode(), capacity);
+ return getIndex(text.getBytes(), 0, text.getLength());
+ }
+
+ /**
+ * Compute the hash value and find the corresponding index. Uses same
+ * implementation as {@code Text#hashCode()}.
+ */
+ int getIndex(final byte[] bytes, final int offset, final int length) {
+ return Math.floorMod(WritableComparator.hashBytes(bytes, offset, length),
Review comment:
Now that there is an implicit dependency on the hash function of
`Test#hashCode` and this class, is there any way to enforce that a change in
the hash function would reflect here as well?
--
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.
To unsubscribe, e-mail: dev-unsubscr...@orc.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
