Skip to content

types: extract month support 0#10116

Merged
zz-jason merged 3 commits intopingcap:masterfrom
WangXiangUSTC:xiang/fix_extract_month
Apr 11, 2019
Merged

types: extract month support 0#10116
zz-jason merged 3 commits intopingcap:masterfrom
WangXiangUSTC:xiang/fix_extract_month

Conversation

@WangXiangUSTC
Copy link
Copy Markdown
Contributor

What problem does this PR solve?

fix extract month in issue #9728
select extract(MONTH FROM "0000-00-00") should return 0

What is changed and how it works?

remove some code, and add unit test

Check List

Tests

  • Unit test

Related changes

  • Need to cherry-pick to the release branch

@qw4990 qw4990 added the type/bugfix This PR fixes a bug. label Apr 11, 2019
@qw4990 qw4990 self-requested a review April 11, 2019 05:21
Copy link
Copy Markdown
Contributor

@qw4990 qw4990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@qw4990 qw4990 requested a review from alivxxx April 11, 2019 07:28
Copy link
Copy Markdown
Contributor

@alivxxx alivxxx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 11, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@4f20b5b). Click here to learn what that means.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master     #10116   +/-   ##
===========================================
  Coverage          ?   78.6088%           
===========================================
  Files             ?        405           
  Lines             ?      83885           
  Branches          ?          0           
===========================================
  Hits              ?      65941           
  Misses            ?      13245           
  Partials          ?       4699

@alivxxx
Copy link
Copy Markdown
Contributor

alivxxx commented Apr 11, 2019

/run-all-tests

@qw4990
Copy link
Copy Markdown
Contributor

qw4990 commented Apr 11, 2019

/rebuild

@zz-jason zz-jason added the status/LGT2 Indicates that a PR has LGTM 2. label Apr 11, 2019
@zz-jason zz-jason merged commit 7423c3c into pingcap:master Apr 11, 2019
@WangXiangUSTC WangXiangUSTC deleted the xiang/fix_extract_month branch April 11, 2019 09:40
qw4990 pushed a commit to qw4990/tidb that referenced this pull request Jun 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/LGT2 Indicates that a PR has LGTM 2. type/bugfix This PR fixes a bug. type/compatibility

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants