[ https://issues.apache.org/jira/browse/HIVE-16890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16274077#comment-16274077 ]
Hive QA commented on HIVE-16890: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12900089/HIVE-16890.1.patch {color:red}ERROR:{color} -1 due to no test(s) being added or modified. {color:red}ERROR:{color} -1 due to 5 failed/errored test(s), 11493 tests executed *Failed tests:* {noformat} org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[vector_complex_join] (batchId=44) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[hybridgrace_hashjoin_2] (batchId=157) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[insert_values_orig_table_use_metadata] (batchId=165) org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[sysdb] (batchId=160) org.apache.hadoop.hive.ql.parse.TestReplicationScenarios.testConstraints (batchId=227) {noformat} Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/8074/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/8074/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-8074/ Messages: {noformat} Executing org.apache.hive.ptest.execution.TestCheckPhase Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.YetusPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase Tests exited with: TestsFailedException: 5 tests failed {noformat} This message is automatically generated. ATTACHMENT ID: 12900089 - PreCommit-HIVE-Build > org.apache.hadoop.hive.serde2.io.HiveVarcharWritable - Adds Superfluous > Wrapper > ------------------------------------------------------------------------------- > > Key: HIVE-16890 > URL: https://issues.apache.org/jira/browse/HIVE-16890 > Project: Hive > Issue Type: Improvement > Components: Serializers/Deserializers > Reporter: BELUGA BEHR > Assignee: BELUGA BEHR > Priority: Trivial > Attachments: HIVE-16890.1.patch, HIVE-16890.1.patch, > HIVE-16890.1.patch > > > Class {{org.apache.hadoop.hive.serde2.io.HiveVarcharWritable}} creates a > superfluous wrapper and then immediately unwraps it. Don't bother wrapping > in this scenario. > {code} > public void set(HiveVarchar val, int len) { > set(val.getValue(), len); > } > public void set(String val, int maxLength) { > value.set(HiveBaseChar.enforceMaxLength(val, maxLength)); > } > public HiveVarchar getHiveVarchar() { > return new HiveVarchar(value.toString(), -1); > } > // Here calls getHiveVarchar() which creates a new HiveVarchar object with > a string in it > // The object is passed to set(HiveVarchar val, int len) > // The string is pulled out > public void enforceMaxLength(int maxLength) { > // Might be possible to truncate the existing Text value, for now just do > something simple. > if (value.getLength()>maxLength && getCharacterLength()>maxLength) > set(getHiveVarchar(), maxLength); > } > {code} -- This message was sent by Atlassian JIRA (v6.4.14#64029)