Hello Philip Zeyliger, Sailesh Mukil, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9957 to look at the new patch set (#3). Change subject: IMPALA-5607: part 1: breaking extract/date_part changes ...................................................................... IMPALA-5607: part 1: breaking extract/date_part changes This is the compatibility-breaking part of Jinchul Kim's change to add additional units. To support nanoseconds we need to widen the output type of these functions. We also change the meaning of "milliseconds" to include the seconds component. Cherry-picks: not for 2.x Change-Id: I42d83712d9bb3a4900bec38a9c009dcf2a1fe019 --- M be/src/exprs/expr-test.cc M be/src/exprs/udf-builtins-ir.cc M be/src/exprs/udf-builtins.h M common/function-registry/impala_functions.py M testdata/workloads/functional-query/queries/QueryTest/aggregation.test M testdata/workloads/functional-query/queries/QueryTest/exprs.test 6 files changed, 94 insertions(+), 86 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/57/9957/3 -- To view, visit http://gerrit.cloudera.org:8080/9957 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I42d83712d9bb3a4900bec38a9c009dcf2a1fe019 Gerrit-Change-Number: 9957 Gerrit-PatchSet: 3 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>