[ https://issues.apache.org/jira/browse/HIVE-15741?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15843169#comment-15843169 ]
Hive QA commented on HIVE-15741: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12849671/HIVE-15741.2.patch {color:red}ERROR:{color} -1 due to no test(s) being added or modified. {color:red}ERROR:{color} -1 due to 27 failed/errored test(s), 11003 tests executed *Failed tests:* {noformat} TestDerbyConnector - did not produce a TEST-*.xml file (likely timed out) (batchId=235) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[vector_udf2] (batchId=11) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[vectorization_10] (batchId=23) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[vectorization_11] (batchId=35) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[vectorization_12] (batchId=10) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[vectorization_15] (batchId=59) org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[vectorization_6] (batchId=25) org.apache.hadoop.hive.cli.TestEncryptedHDFSCliDriver.testCliDriver[encryption_join_with_different_encryption_keys] (batchId=159) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vector_char_simple] (batchId=147) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vector_if_expr] (batchId=140) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vectorization_0] (batchId=154) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vectorization_12] (batchId=140) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vectorization_15] (batchId=151) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vectorization_short_regress] (batchId=148) org.apache.hadoop.hive.cli.TestMiniTezCliDriver.testCliDriver[explainuser_3] (batchId=93) org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[vectorization_0] (batchId=130) org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[vectorization_12] (batchId=99) org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[vectorization_15] (batchId=122) org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[vectorization_short_regress] (batchId=115) org.apache.hadoop.hive.common.type.TestHiveDecimal.testBigIntegerBytesScaledSpecial (batchId=169) org.apache.hadoop.hive.common.type.TestHiveDecimal.testBigIntegerBytesSpecial (batchId=169) org.apache.hadoop.hive.common.type.TestHiveDecimal.testRandomBigIntegerBytes (batchId=169) org.apache.hadoop.hive.common.type.TestHiveDecimal.testRandomBigIntegerBytesScaled (batchId=169) org.apache.hadoop.hive.ql.exec.vector.expressions.TestVectorStringExpressions.testStringLike (batchId=266) org.apache.hadoop.hive.ql.exec.vector.expressions.TestVectorStringExpressions.testStringLikeRandomized (batchId=266) org.apache.hive.hcatalog.pig.TestTextFileHCatStorer.testWriteChar (batchId=173) org.apache.hive.hcatalog.pig.TestTextFileHCatStorer.testWriteDate (batchId=173) {noformat} Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/3218/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/3218/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-3218/ Messages: {noformat} Executing org.apache.hive.ptest.execution.TestCheckPhase Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase Tests exited with: TestsFailedException: 27 tests failed {noformat} This message is automatically generated. ATTACHMENT ID: 12849671 - PreCommit-HIVE-Build > Faster unsafe byte array comparisons > ------------------------------------ > > Key: HIVE-15741 > URL: https://issues.apache.org/jira/browse/HIVE-15741 > Project: Hive > Issue Type: Improvement > Reporter: Teddy Choi > Assignee: Teddy Choi > Priority: Minor > Attachments: HIVE-15741.1.patch, HIVE-15741.2.patch > > > Byte array comparison is heavily used in joins and string conditions. Pure > Java implementation is simple but not performant. An implementation with > Unsafe#getLong is much faster. It's already implemented in > org.apache.hadoop.io.WritableComparator#compare. The WritableComparator class > handles exceptional cases, including a different endian and no access to > Unsafe, and it was used for many years in production. > This patch will replace pure Java byte array comparisons with safe and faster > unsafe ones to get more performance. -- This message was sent by Atlassian JIRA (v6.3.4#6332)