Github user kiszk commented on a diff in the pull request: https://github.com/apache/spark/pull/20568#discussion_r167419960 --- Diff: common/unsafe/src/test/java/org/apache/spark/unsafe/hash/Murmur3_x86_32Suite.java --- @@ -51,6 +51,22 @@ public void testKnownLongInputs() { Assert.assertEquals(-2106506049, hasher.hashLong(Long.MAX_VALUE)); } + @Test --- End diff -- It would be good to add JIRA number with a short description as a comment (e.g. `SPARK-23381 ...`)
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org