Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/14848 )

Change subject: IMPALA-9217: Adjust limits for TZH and TZM datetime tokens
......................................................................

IMPALA-9217: Adjust limits for TZH and TZM datetime tokens

Values for TZH are expected in a range of [-15, 15] and for TZM are
expected in [0, 59]. This patch adjusts the checks for these limits
and adds some tests to verify that out of range values are not
accepted.

Change-Id: I6fe2534d51396bb5652af6301866e2dd0f3282c2
Reviewed-on: http://gerrit.cloudera.org:8080/14848
Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
---
M be/src/runtime/datetime-iso-sql-format-parser.cc
M tests/query_test/test_cast_with_format.py
2 files changed, 19 insertions(+), 2 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I6fe2534d51396bb5652af6301866e2dd0f3282c2
Gerrit-Change-Number: 14848
Gerrit-PatchSet: 3
Gerrit-Owner: Gabor Kaszab <gaborkas...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward <miko.szil...@gmail.com>
Gerrit-Reviewer: Attila Jeges <atti...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Norbert Luksa <norbert.lu...@cloudera.com>

Reply via email to