-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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 checkMonthDay() to correctly handle leap year #10342
Conversation
Codecov Report
@@ Coverage Diff @@
## master #10342 +/- ##
===============================================
- Coverage 77.6767% 77.6578% -0.019%
===============================================
Files 411 411
Lines 85440 85390 -50
===============================================
- Hits 66367 66312 -55
- Misses 14112 14116 +4
- Partials 4961 4962 +1 |
Codecov Report
@@ Coverage Diff @@
## master #10342 +/- ##
==============================================
+ Coverage 77.6662% 77.6762% +0.01%
==============================================
Files 411 411
Lines 85440 85389 -51
==============================================
- Hits 66358 66327 -31
+ Misses 14118 14103 -15
+ Partials 4964 4959 -5 |
/rebuild |
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 |
@erjiaqing please cherry pick this bug-fix to release-2.1 |
What problem does this PR solve?
1900-02-29 is not a valid date. However:
In TiDB:
MySQL does not consider 1900 as a leap year. So this is not intended.
In MySQL:
What is changed and how it works?
Check List
Tests