Skip to content

Conversation

@zclllyybb
Copy link
Contributor

pick #51454

Problem Summary:

before:
```sql
mysql> select/*+SET_VAR(debug_skip_fold_constant=true)*/ cast('2010-01-05 08:15:30Europe/London' as datetime);
+------------------------------------------------------+
| cast('2010-01-05 08:15:30Europe/London' as datetime) |
+------------------------------------------------------+
| 2010-01-05 15:15:30                                  |
+------------------------------------------------------+
1 row in set (0.19 sec)
```

now:
```sql
mysql> select/*+SET_VAR(debug_skip_fold_constant=true)*/ cast('2010-01-05 08:15:30Europe/London' as datetime);
+------------------------------------------------------+
| cast('2010-01-05 08:15:30Europe/London' as datetime) |
+------------------------------------------------------+
| 2010-01-05 16:15:30                                  |
+------------------------------------------------------+
1 row in set (0.12 sec)
```
@zclllyybb zclllyybb requested a review from morrySnow as a code owner June 27, 2025 06:29
@zclllyybb zclllyybb changed the title [Fix](timezone) Fix incorrect DST handling (#51454) [branch-3.1](timezone) Fix incorrect DST handling (#51454) Jun 27, 2025
@zclllyybb
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

BE UT Coverage Report

Increment line coverage 20.00% (6/30) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 44.99% (12340/27428)
Line Coverage 36.01% (109297/303533)
Region Coverage 35.05% (56628/161561)
Branch Coverage 32.16% (30712/95486)

@morrySnow morrySnow changed the title [branch-3.1](timezone) Fix incorrect DST handling (#51454) branch-3.1: [fix](timezone) Fix incorrect DST handling (#51454) Jun 28, 2025
@morrySnow morrySnow changed the title branch-3.1: [fix](timezone) Fix incorrect DST handling (#51454) branch-3.1: [fix](timezone) Fix incorrect DST handling #51454 Jun 28, 2025
@morrySnow morrySnow merged commit 1d8f09a into apache:branch-3.1 Jun 28, 2025
20 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants