Peter Rozsa has uploaded a new patch set (#10). ( http://gerrit.cloudera.org:8080/19096 )
Change subject: IMPALA-11537: Query option validation for numeric types ...................................................................... IMPALA-11537: Query option validation for numeric types This change adds a more generic approach to validate numeric query options and report parse and validation errors. Supported types: integers, floats, memory specifications. Range and bound validator helper functions are added to make validation unified on call sites. Testing: - Error messages got more generic, therefore the existing tests around query options are aligned to match them Change-Id: Ia7757b52393c094d2c661918d73cbfad7214f855 --- A be/src/service/query-option-parser.h M be/src/service/query-options.cc M be/src/util/parse-util.h M common/thrift/generate_error_codes.py M testdata/workloads/functional-query/queries/QueryTest/set.test 5 files changed, 614 insertions(+), 568 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/96/19096/10 -- To view, visit http://gerrit.cloudera.org:8080/19096 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia7757b52393c094d2c661918d73cbfad7214f855 Gerrit-Change-Number: 19096 Gerrit-PatchSet: 10 Gerrit-Owner: Peter Rozsa <pro...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Gergely Fürnstáhl <gfurnst...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Peter Rozsa <pro...@cloudera.com>