Yifan Zhang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21441 )

Change subject: IMPALA-12562: Cast double and float to string with exact 
presicion
......................................................................


Patch Set 2:

(1 comment)

> Patch Set 1: Code-Review+1
>
> (1 comment)
>
> Thanks for fixing this!

Thanks for your review!

http://gerrit.cloudera.org:8080/#/c/21441/1/testdata/workloads/functional-query/queries/QueryTest/exprs.test
File testdata/workloads/functional-query/queries/QueryTest/exprs.test:

http://gerrit.cloudera.org:8080/#/c/21441/1/testdata/workloads/functional-query/queries/QueryTest/exprs.test@3300
PS1, Line 3300: select cast(round(cast(1.33 as double), 2) as string);
> A few more test cases around the limits of large/small number of decimals w
Done



--
To view, visit http://gerrit.cloudera.org:8080/21441
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icd79c55dd57dc0fa13e4ec11c2284ef2800e8b1a
Gerrit-Change-Number: 21441
Gerrit-PatchSet: 2
Gerrit-Owner: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Gabor Kaszab <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>
Gerrit-Comment-Date: Sat, 18 May 2024 01:14:15 +0000
Gerrit-HasComments: Yes

Reply via email to