Noemi Pap-Takacs has uploaded a new patch set (#2). ( 
http://gerrit.cloudera.org:8080/24571 )

Change subject: IMPALA-13953: Remove deprecated flag 
use_local_tz_for_unix_timestamp_conversions
......................................................................

IMPALA-13953: Remove deprecated flag use_local_tz_for_unix_timestamp_conversions

IMPALA-10171 added USE_LOCAL_TZ_FOR_UNIX_TIMESTAMP_CONVERSIONS
query option to override backend flag with the same name.
This change moved the startup flag to the graveyard and updated
the documentation to use the query option instead.

Deprecated the convert_legacy_hive_parquet_utc_timestamps
startup flag as it also has a corresponding query option that
overrides its value. See IMPALA-10171.

Testing:
 - removed tests/custom_cluster/test_local_tz_conversion.py
   because its tests are all covered by test_exprs.py
 - added disable_codegen test dimension to test_exprs.py

Assisted-by: Claude Opus 4.6
Change-Id: I1245783ba45a42f02f649ac122c1ce74938f5141
---
M be/src/common/global-flags.cc
M be/src/exec/data-source-scan-node.cc
M be/src/service/impala-server.cc
M common/thrift/ImpalaService.thrift
M docs/shared/impala_common.xml
M docs/topics/impala_datetime_functions.xml
M docs/topics/impala_new_features.xml
M docs/topics/impala_timestamp.xml
M docs/topics/impala_timezone.xml
M tests/custom_cluster/test_kudu.py
D tests/custom_cluster/test_local_tz_conversion.py
M tests/query_test/test_exprs.py
M tests/query_test/test_kudu.py
13 files changed, 42 insertions(+), 117 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/71/24571/2
--
To view, visit http://gerrit.cloudera.org:8080/24571
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1245783ba45a42f02f649ac122c1ce74938f5141
Gerrit-Change-Number: 24571
Gerrit-PatchSet: 2
Gerrit-Owner: Noemi Pap-Takacs <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Daniel Vanko <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Mihaly Szjatinya <[email protected]>
Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>

Reply via email to