OIiveirra opened a new pull request, #68864:
URL: https://github.com/apache/doris/pull/68864
### What problem does this PR solve?
Jira: DORIS-29598
External regression goldens on branch-4.1 still expect wall-clock or string
output for columns now mapped to TIMESTAMPTZ. Update OceanBase TIMESTAMP, SQL
Server DATETIMEOFFSET (including its CTAS type and session-zone conversion),
MaxCompute TIMESTAMP, and Hudi timestamp goldens. Preserve DATETIME and
TIMESTAMP_NTZ expectations. Set the session timezone explicitly where the
goldens depend on Asia/Shanghai.
HTTP Parquet TVF queries sql08/sql09/sql10 now specify
`hive.parquet.time-zone=Asia/Shanghai` to make their existing expectations
deterministic.
```diff
- instant columns: old LocalDateTime/string expectations
+ instant columns: TIMESTAMPTZ precision and session-zone expectations
wall-clock columns: existing expectations retained
```
### Validation
- Before: branch-4.1 external CI build 215484 reported timestamp output
mismatches in the five affected external suites.
- After: local `test_http_tvf` (`-g p2 -d external_table_p0/tvf -s
test_http_tvf`) passed: 1 suite, 0 failures, 0 skips, including
sql08/sql09/sql10.
- Local SQL verified SQL Server's source `2023-06-25 14:30:45.123456-07:00`
renders as `2023-06-26 05:30:45.123456+08:00`, and CTAS preserves
`timestamptz(6)`.
- `git diff --check` passed. Local Doris revision: `6ab06fceef9`; this is
not an exact branch-4.1 end-to-end result.
- JDBC/Hudi fixtures and MaxCompute cloud prerequisites are unavailable in
the personal environment. Their full-suite runtime validation remains pending
community CI; their goldens were reviewed against branch-4.1 mappings and the
existing CI mismatch evidence.
### Release note
None
### Check List (For Author)
- Test
- [x] Regression test (HTTP TVF locally; external suite coverage pending
CI)
- [x] Manual test (local timestamp conversion and CTAS type probe
described above)
- Behavior changed:
- [x] No production behavior change; regression expectations and
timezone setup only.
- Does this need documentation?
- [x] No.
### Merge Danger
Two-way door; regression-only changes can be reverted. Blast radius:
regression.
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
