[jira] [Commented] (HIVE-10657) Remove copyBytes operation from MD5 UDF

2015-05-13 Thread Jason Dere (JIRA) 

[ 
https://issues.apache.org/jira/browse/HIVE-10657?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14543060#comment-14543060
 ] 

Jason Dere commented on HIVE-10657:
---

+1

 Remove copyBytes operation from MD5 UDF
 ---

 Key: HIVE-10657
 URL: https://issues.apache.org/jira/browse/HIVE-10657
 Project: Hive
  Issue Type: Improvement
  Components: UDF
Reporter: Alexander Pivovarov
Assignee: Alexander Pivovarov
Priority: Minor
 Attachments: HIVE-10657.1.patch, HIVE-10657.2.patch


 Current MD5 UDF implementation uses Apache Commons  DigestUtils.md5Hex method 
 to get md5 hex.
 DigestUtils does not provide md5Hex method with signature (byte[], start, 
 length). This is why copyBytes method was added to UDFMd5 to get bytes[] from 
 BytesWritable.
 To avoid copying bytes from BytesWritable to new byte array we can use java 
 MessageDigest API directly.
 MessageDigest has method update(byte[], start, length)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (HIVE-10657) Remove copyBytes operation from MD5 UDF

2015-05-13 Thread Hive QA (JIRA) 

[ 
https://issues.apache.org/jira/browse/HIVE-10657?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14541518#comment-14541518
 ] 

Hive QA commented on HIVE-10657:




{color:red}Overall{color}: -1 at least one tests failed

Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12732393/HIVE-10657.2.patch

{color:red}ERROR:{color} -1 due to 1 failed/errored test(s), 8921 tests executed
*Failed tests:*
{noformat}
org.apache.hadoop.hive.cli.TestEncryptedHDFSCliDriver.testCliDriver_encryption_insert_partition_static
{noformat}

Test results: 
http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/3870/testReport
Console output: 
http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/3870/console
Test logs: 
http://ec2-174-129-184-35.compute-1.amazonaws.com/logs/PreCommit-HIVE-TRUNK-Build-3870/

Messages:
{noformat}
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: 1 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12732393 - PreCommit-HIVE-TRUNK-Build

 Remove copyBytes operation from MD5 UDF
 ---

 Key: HIVE-10657
 URL: https://issues.apache.org/jira/browse/HIVE-10657
 Project: Hive
  Issue Type: Improvement
  Components: UDF
Reporter: Alexander Pivovarov
Assignee: Alexander Pivovarov
Priority: Minor
 Attachments: HIVE-10657.1.patch, HIVE-10657.2.patch


 Current MD5 UDF implementation uses Apache Commons  DigestUtils.md5Hex method 
 to get md5 hex.
 DigestUtils does not provide md5Hex method with signature (byte[], start, 
 length). This is why copyBytes method was added to UDFMd5 to get bytes[] from 
 BytesWritable.
 To avoid copying bytes from BytesWritable to new byte array we can use java 
 MessageDigest API directly.
 MessageDigest has method update(byte[], start, length)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (HIVE-10657) Remove copyBytes operation from MD5 UDF

2015-05-09 Thread Hive QA (JIRA) 

[ 
https://issues.apache.org/jira/browse/HIVE-10657?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14536840#comment-14536840
 ] 

Hive QA commented on HIVE-10657:




{color:red}Overall{color}: -1 at least one tests failed

Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12731545/HIVE-10657.1.patch

{color:red}ERROR:{color} -1 due to 4 failed/errored test(s), 8920 tests executed
*Failed tests:*
{noformat}
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver_udaf_percentile_approx_23
org.apache.hadoop.hive.cli.TestEncryptedHDFSCliDriver.testCliDriver_encryption_insert_partition_static
org.apache.hadoop.hive.thrift.TestHadoop20SAuthBridge.testSaslWithHiveMetaStore
org.apache.hive.jdbc.TestSSL.testSSLFetchHttp
{noformat}

Test results: 
http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/3830/testReport
Console output: 
http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/3830/console
Test logs: 
http://ec2-174-129-184-35.compute-1.amazonaws.com/logs/PreCommit-HIVE-TRUNK-Build-3830/

Messages:
{noformat}
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: 4 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12731545 - PreCommit-HIVE-TRUNK-Build

 Remove copyBytes operation from MD5 UDF
 ---

 Key: HIVE-10657
 URL: https://issues.apache.org/jira/browse/HIVE-10657
 Project: Hive
  Issue Type: Improvement
  Components: UDF
Reporter: Alexander Pivovarov
Assignee: Alexander Pivovarov
Priority: Minor
 Attachments: HIVE-10657.1.patch


 Current MD5 UDF implementation uses Apache Commons  DigestUtils.md5Hex method 
 to get md5 hex.
 DigestUtils does not provide md5Hex method with signature (byte[], start, 
 length). This is why copyBytes method was added to UDFMd5 to get bytes[] from 
 BytesWritable.
 To avoid copying bytes from BytesWritable to new byte array we can use java 
 MessageDigest API directly.
 MessageDigest has method update(byte[], start, length)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)