[ 
https://issues.apache.org/jira/browse/HIVE-4174?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Phabricator updated HIVE-4174:
------------------------------

    Attachment: HIVE-4174.D9687.1.patch

chenchun requested code review of "HIVE-4174 [jira] Round UDF converts BigInts 
to double".

Reviewers: JIRA

Round UDF return type be the same type as that of the first argument

Chen Chun pointed out on the hive-user mailing list that round() in Hive 0.10 
returns

select round(cast(1234560 as BIGINT)), round(cast(12345670 as BIGINT)) from 
test limit 1;

//hive 0.10
1234560.0  1.234567E7

This is not consistent with 
MySQL(http://dev.mysql.com/doc/refman/5.1/en/mathematical-functions.html#function_round)

which quotes

The return type is the same type as that of the first argument (assuming that 
it is integer, double, or decimal). This means that for an integer argument, 
the result is an integer (no decimal places)

TEST PLAN
  EMPTY

REVISION DETAIL
  https://reviews.facebook.net/D9687

AFFECTED FILES
  ql/src/java/org/apache/hadoop/hive/ql/udf/UDFRound.java
  ql/src/test/queries/clientpositive/udf_round_3.q
  ql/src/test/results/clientpositive/udf_round.q.out
  ql/src/test/results/clientpositive/udf_round_3.q.out

MANAGE HERALD RULES
  https://reviews.facebook.net/herald/view/differential/

WHY DID I GET THIS EMAIL?
  https://reviews.facebook.net/herald/transcript/23127/

To: JIRA, chenchun

                
> Round UDF converts BigInts to double
> ------------------------------------
>
>                 Key: HIVE-4174
>                 URL: https://issues.apache.org/jira/browse/HIVE-4174
>             Project: Hive
>          Issue Type: Bug
>          Components: UDF
>    Affects Versions: 0.10.0
>            Reporter: Mark Grover
>            Assignee: Mark Grover
>             Fix For: 0.11.0
>
>         Attachments: HIVE-4174.1.patch.txt, HIVE-4174.D9687.1.patch
>
>
> Chen Chun pointed out on the hive-user mailing list that round() in Hive 0.10 
> returns
> {code}
> select round(cast(1234560 as BIGINT)), round(cast(12345670 as BIGINT)) from 
> test limit 1;
> //hive 0.10
> 1234560.0  1.234567E7
> {code}
> This is not consistent with 
> MySQL(http://dev.mysql.com/doc/refman/5.1/en/mathematical-functions.html#function_round)
> which quotes
> {code}
> The return type is the same type as that of the first argument (assuming that 
> it is integer, double, or decimal). This means that for an integer argument, 
> the result is an integer (no decimal places)
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to