-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
types: fix incorrect time fraction parsing method #9933
Conversation
/run-all-tests |
/rebuild |
/run-all-tests |
Any reference mysql doc? |
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
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
/run-all-tests |
Codecov Report
@@ Coverage Diff @@
## master #9933 +/- ##
================================================
+ Coverage 77.4719% 77.7943% +0.3224%
================================================
Files 403 403
Lines 81809 82479 +670
================================================
+ Hits 63379 64164 +785
+ Misses 13711 13631 -80
+ Partials 4719 4684 -35 |
@winkyao I search the mysql's document, can't find the detail about the format of date time string. I just execute in mysql to check. |
@WangXiangUSTC Please cherry pick this PR to release 2.1 |
What problem does this PR solve?
fix issue #9734
tidb don't support date time format like "2019.01.01"
What is changed and how it works?
some minor update, and add unit test
Check List
Tests
Related changes