-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[feature](timestamptz) support mapping timestamp_tz type in external catalog #59720
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
TPC-H: Total hot run time: 32303 ms |
TPC-DS: Total hot run time: 172484 ms |
d9a35f9 to
0179489
Compare
|
run buildall |
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
0179489 to
3f83aab
Compare
|
run buildall |
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
TPC-H: Total hot run time: 32352 ms |
TPC-DS: Total hot run time: 173357 ms |
FE UT Coverage ReportIncrement line coverage |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
3f83aab to
093e1e9
Compare
|
run buildall |
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
FE UT Coverage ReportIncrement line coverage |
TPC-H: Total hot run time: 31878 ms |
TPC-DS: Total hot run time: 172742 ms |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
|
run buildall |
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
TPC-H: Total hot run time: 31851 ms |
TPC-DS: Total hot run time: 173013 ms |
ClickBench: Total hot run time: 27.8 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
FE UT Coverage ReportIncrement line coverage |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
...er/thirdparties/docker-compose/iceberg/scripts/create_preinstalled_scripts/iceberg/run25.sql
Outdated
Show resolved
Hide resolved
docker/thirdparties/docker-compose/iceberg/scripts/create_preinstalled_scripts/paimon/run10.sql
Show resolved
Hide resolved
regression-test/suites/external_table_p0/iceberg/test_iceberg_export_timestamp_tz.groovy
Outdated
Show resolved
Hide resolved
|
run buildall |
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
d4c6928 to
de5c8fa
Compare
|
run buildall |
What problem does this PR solve?
Problem Summary:
before the type of timestamp with local time zone will be mapping to doris datetime type, because doris don't have a type with time zone, But now doris have support timestamp_tz type, which have the time zone info.
so change the timestamp mapping in some catalog.
add a conf property in create catalog "enable.mapping.timestamp_tz", default is false, so mapping TIMESTAMP_TZ types to datetime for compatibility, if it's set to true, will mapping the type to doris timestamp_tz. more info could ref https://doris.apache.org/zh-CN/docs/dev/sql-manual/basic-element/sql-data-types/date-time/TIMESTAMPTZ
doc: apache/doris-website#3285
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)