impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matthew Jacobs (Code Review)" <>
Subject [Impala-CR](cdh5-2.6.0_5.8.0) IMPALA-3579: Strict handling of numeric overflow in text parsing
Date Fri, 20 May 2016 22:51:21 GMT
Hello Dan Hecht,

I'd like you to reexamine a change.  Please visit

to look at the new patch set (#4).

Change subject: IMPALA-3579: Strict handling of numeric overflow in text parsing

IMPALA-3579: Strict handling of numeric overflow in text parsing

Adds a query option 'strict_mode' which treats integer and
floating pt overflows as parse errors. In the past,
overflows were ignored and the max value was returned. When
this query option is set, overflowing values are treated as if
they were completely invalid data, i.e. NULL is returned.
When abort_on_error is enabled, this means the query is

* DECIMAL overflow/underflow is already treated as an error.
* The handling in text-converter treats underflows the same
  as overflows, so they would result in the same behavior.
  However, floating point parsing never returns an underflow
* We may also want to handle numeric values that are truncated
  when parsing to integer types, e.g. 10.5 -> 10.

Change-Id: I7409c31ec0cb6fe0b2d9842b9f58fe1670914836
M be/src/exec/
M be/src/exec/
M be/src/exec/
M be/src/exec/text-converter.h
M be/src/exec/text-converter.inline.h
M be/src/runtime/runtime-state.h
M be/src/service/
M be/src/service/query-options.h
M common/thrift/ImpalaInternalService.thrift
M common/thrift/ImpalaService.thrift
M testdata/datasets/functional/functional_schema_template.sql
M testdata/datasets/functional/schema_constraints.csv
M testdata/workloads/functional-query/queries/QueryTest/show.test
A testdata/workloads/functional-query/queries/QueryTest/strict-mode-abort.test
A testdata/workloads/functional-query/queries/QueryTest/strict-mode.test
M tests/query_test/
16 files changed, 132 insertions(+), 22 deletions(-)

  git pull ssh:// refs/changes/50/3150/4
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7409c31ec0cb6fe0b2d9842b9f58fe1670914836
Gerrit-PatchSet: 4
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.6.0_5.8.0
Gerrit-Owner: Matthew Jacobs <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: Matthew Jacobs <>

View raw message