Adam Tamas has uploaded a new patch set (#8). ( )

Change subject: IMPALA-9531: Dropped support for dateless timestamps

IMPALA-9531: Dropped support for dateless timestamps

Removed the support for dateless timestamps.
During dateless timestamp casts if the format doesn't contain
date part we get an error during tokenization of the format.
If the input str doesn't contain a date part then we get null result.

select cast('01:02:59' as timestamp);
This will come back as NULL value.

select to_timestamp('01:01:01', 'HH:mm:ss');
select cast('01:02:59' as timestamp format 'HH12:MI:SS');
select cast('12 AM' as timestamp FORMAT 'AM.HH12');
These will come back with a parsing errors.

Casting from a table will generate similar results.

Modified the previous tests related to dateless timestamps.
Added test to read fromtables which are still containing dateless
timestamps and covered timestamp to string path when no date tokens
are requested in the output string.

Change-Id: I48c49bf027cc4b917849b3d58518facba372b322
M be/src/benchmarks/
M be/src/benchmarks/
M be/src/exec/text-converter.inline.h
M be/src/exprs/
M be/src/exprs/
M be/src/exprs/
M be/src/exprs/
M be/src/exprs/
M be/src/exprs/timestamp-functions.h
M be/src/runtime/
M be/src/runtime/
M be/src/runtime/
M be/src/runtime/
M be/src/runtime/datetime-parser-common.h
M be/src/runtime/
M be/src/runtime/datetime-simple-date-format-parser.h
M be/src/runtime/
M be/src/runtime/
M be/src/runtime/timestamp-value.h
M bin/rat_exclude_files.txt
M common/function-registry/
M fe/src/test/java/org/apache/impala/analysis/
M testdata/data/README
A testdata/data/dateless_timestamps.parq
A testdata/data/dateless_timestamps.txt
M testdata/data/lazy_timestamp.csv
M testdata/workloads/functional-query/queries/QueryTest/date.test
M testdata/workloads/functional-query/queries/QueryTest/exprs.test
M tests/data_errors/
M tests/query_test/
M tests/query_test/
34 files changed, 278 insertions(+), 231 deletions(-)

  git pull ssh:// refs/changes/66/15866/8
To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I48c49bf027cc4b917849b3d58518facba372b322
Gerrit-Change-Number: 15866
Gerrit-PatchSet: 8
Gerrit-Owner: Adam Tamas <>
Gerrit-Reviewer: Adam Tamas <>
Gerrit-Reviewer: Gabor Kaszab <>
Gerrit-Reviewer: Impala Public Jenkins <>

Reply via email to