-
Notifications
You must be signed in to change notification settings - Fork 205
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
fix: Parse string without specify time zone to local time stamp #692
Conversation
Codecov Report
@@ Coverage Diff @@
## main #692 +/- ##
=======================================
Coverage 67.55% 67.55%
=======================================
Files 281 281
Lines 44301 44343 +42
=======================================
+ Hits 29926 29955 +29
- Misses 14375 14388 +13
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
374c586
to
d14b6ca
Compare
47c31b4
to
9dd393d
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.
LGTM
Which issue does this PR close?
Closes #
Rationale for this change
I found the following query response nothing :
Actually it should response:
And I found the reason is , according to apache/arrow-rs#2814, function
string_to_timestamp_nanos
returns a UTC timestamp instead of local timestamp;What changes are included in this PR?
string_to_timestamp_nanos
to parse it to utc time stampAre there any user-facing changes?
How does this change test
By unit test