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
feat: make query be aware of timezone setting #3175
feat: make query be aware of timezone setting #3175
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3175 +/- ##
==========================================
- Coverage 85.81% 85.35% -0.46%
==========================================
Files 833 840 +7
Lines 137109 137929 +820
==========================================
+ Hits 117662 117732 +70
- Misses 19447 20197 +750 |
4358217
to
c966468
Compare
I'll rebase #3193 after it is merged. |
c966468
to
8cf131e
Compare
0a00d8a
to
43f1003
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Others LGTM, except the from/to local tz
Co-authored-by: Lei, HUANG <6406592+v0y4g3r@users.noreply.github.com>
Renamed the |
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Try to aware of timezone setting for
TypeConversionRule
, changes:ExtensionAnalyzerRule
that extends DataFusionAnalyzerRule
to acceptQueryEngineContext
as the second parameter.ExtensionAnalyzerRule
forTypeConversionRule
.DfLogicalPlanner
andLogicalOptimizer
.show variables time_zone
can't work.Date
,Timestamp
andDateTime
parsing from string value aware of timezone.Checklist
Refer to a related PR or issue link (optional)
#2907