Hello Tim Armstrong, Bikramjeet Vig, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15864
to look at the new patch set (#2).
Change subject: IMPALA-7833 Audit and fix string builtins for long string
handling
......................................................................
IMPALA-7833 Audit and fix string builtins for long string handling
Some string built-in functions could crash impalad,
in case the result was longer than 1 gig max size.
Added some overflow checks.
Testing:
* Added some backend tests to cover overflow check
* Ran core tests
Change-Id: I93a53845f04e61ff446b363c78db1e49cbd5dc49
---
M be/src/exprs/string-functions-ir.cc
M testdata/workloads/functional-query/queries/QueryTest/large_strings.test
2 files changed, 67 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/64/15864/2
--
To view, visit http://gerrit.cloudera.org:8080/15864
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I93a53845f04e61ff446b363c78db1e49cbd5dc49
Gerrit-Change-Number: 15864
Gerrit-PatchSet: 2
Gerrit-Owner: Akos Kovacs <akov...@cloudera.com>
Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
