Xiang Yang has posted comments on this change. ( http://gerrit.cloudera.org:8080/18861 )
Change subject: IMPALA-11504: Specializing DecimalUtil::GetScaleMultiplier<int256_t>(). ...................................................................... Patch Set 9: (3 comments) http://gerrit.cloudera.org:8080/#/c/18861/8/be/src/benchmarks/decimal-util-benchmark.cc File be/src/benchmarks/decimal-util-benchmark.cc: http://gerrit.cloudera.org:8080/#/c/18861/8/be/src/benchmarks/decimal-util-benchmark.cc@25 PS8, Line 25: (ra > Nit: ran, no need for capital letter. Done http://gerrit.cloudera.org:8080/#/c/18861/8/be/src/benchmarks/decimal-util-benchmark.cc@25 PS8, Line 25: // Summary result (ran within a Docker container): > Nit: add a space before the parenthesis. Done http://gerrit.cloudera.org:8080/#/c/18861/8/be/src/benchmarks/decimal-util-benchmark.cc@54 PS8, Line 54: // A volatile variable to hold the return value of the benchmarked functions. > I think this would be better/more understantable: Done -- To view, visit http://gerrit.cloudera.org:8080/18861 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I969e2977d51313e738f72c8246db003ae43a3782 Gerrit-Change-Number: 18861 Gerrit-PatchSet: 9 Gerrit-Owner: Xiang Yang <yx91...@126.com> Gerrit-Reviewer: Anonymous Coward <lipeng...@sensorsdata.cn> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Xiang Yang <yx91...@126.com> Gerrit-Comment-Date: Sat, 08 Oct 2022 02:46:27 +0000 Gerrit-HasComments: Yes