[jira] [Commented] (HIVE-9956) use BigDecimal.valueOf instead of new in TestFileDump

2015-03-18 Thread Hive QA (JIRA)

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

Hive QA commented on HIVE-9956:
---



{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/12705278/HIVE-9956.1.patch

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

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

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: 12705278 - PreCommit-HIVE-TRUNK-Build

 use BigDecimal.valueOf instead of new in TestFileDump
 -

 Key: HIVE-9956
 URL: https://issues.apache.org/jira/browse/HIVE-9956
 Project: Hive
  Issue Type: Bug
  Components: File Formats
Reporter: Alexander Pivovarov
Assignee: Alexander Pivovarov
Priority: Minor
 Attachments: HIVE-9956.1.patch, HIVE-9956.1.patch


 TestFileDump builds data row where one of the column is BigDecimal
 The test adds value 2.
 There are 2 ways to create BigDecimal object.
 1. use new
 2. use valueOf
 in this particular case 
 1. new will create 2.222153
 2. valueOf will use the canonical String representation and the result will 
 be 2.
 Probably we should use valueOf to create BigDecimal object
 TestTimestampWritable and TestHCatStores use valueOf



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


[jira] [Commented] (HIVE-9956) use BigDecimal.valueOf instead of new in TestFileDump

2015-03-18 Thread Alexander Pivovarov (JIRA)

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

Alexander Pivovarov commented on HIVE-9956:
---

I think both failed tests have no relation to patch #1.

 use BigDecimal.valueOf instead of new in TestFileDump
 -

 Key: HIVE-9956
 URL: https://issues.apache.org/jira/browse/HIVE-9956
 Project: Hive
  Issue Type: Bug
  Components: File Formats
Reporter: Alexander Pivovarov
Assignee: Alexander Pivovarov
Priority: Minor
 Attachments: HIVE-9956.1.patch, HIVE-9956.1.patch


 TestFileDump builds data row where one of the column is BigDecimal
 The test adds value 2.
 There are 2 ways to create BigDecimal object.
 1. use new
 2. use valueOf
 in this particular case 
 1. new will create 2.222153
 2. valueOf will use the canonical String representation and the result will 
 be 2.
 Probably we should use valueOf to create BigDecimal object
 TestTimestampWritable and TestHCatStores use valueOf



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


[jira] [Commented] (HIVE-9956) use BigDecimal.valueOf instead of new in TestFileDump

2015-03-13 Thread Xuefu Zhang (JIRA)

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

Xuefu Zhang commented on HIVE-9956:
---

+1

 use BigDecimal.valueOf instead of new in TestFileDump
 -

 Key: HIVE-9956
 URL: https://issues.apache.org/jira/browse/HIVE-9956
 Project: Hive
  Issue Type: Bug
  Components: File Formats
Reporter: Alexander Pivovarov
Assignee: Alexander Pivovarov
Priority: Minor
 Attachments: HIVE-9956.1.patch


 TestFileDump builds data row where one of the column is BigDecimal
 The test adds value 2.
 There are 2 ways to create BigDecimal object.
 1. use new
 2. use valueOf
 in this particular case 
 1. new will create 2.222153
 2. valueOf will use the canonical String representation and the result will 
 be 2.
 Probably we should use valueOf to create BigDecimal object
 TestTimestampWritable and TestHCatStores use valueOf



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