Skip to content
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

expression: keep xor result consistent with mysql 8 #15934

Merged
merged 7 commits into from
May 6, 2020
Merged

expression: keep xor result consistent with mysql 8 #15934

merged 7 commits into from
May 6, 2020

Conversation

sduzh
Copy link
Contributor

@sduzh sduzh commented Mar 31, 2020

What problem does this PR solve?

The 'xor' result is different between tidb and mysql8

Issue Number: close #12867

Problem Summary:

What is changed and how it works?

What's Changed:

Keep xor result consistent with MySQL 8, instead of MySQL 5.7

Related changes

Check List

Tests

  • Unit test
  • Integration test

Side effects

  • Breaking backward compatibility

Release note

  • keep xor results of float arguments consistent with mysql 8

@sduzh sduzh requested a review from a team as a code owner March 31, 2020 15:02
@sre-bot sre-bot added the contribution This PR is from a community contributor. label Mar 31, 2020
@ghost ghost requested review from wshwsh12 and removed request for a team March 31, 2020 15:02
@sduzh sduzh changed the title expression: keep xor result consistent with mysql 8 (#12867) expression: keep xor result consistent with mysql 8 Mar 31, 2020
Copy link
Contributor

@wshwsh12 wshwsh12 left a comment

Choose a reason for hiding this comment

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

LGTM

@wshwsh12 wshwsh12 added the status/LGT1 Indicates that a PR has LGTM 1. label Apr 1, 2020
@wshwsh12 wshwsh12 requested a review from SunRunAway April 1, 2020 03:16
SunRunAway
SunRunAway previously approved these changes Apr 1, 2020
Copy link
Contributor

@SunRunAway SunRunAway left a comment

Choose a reason for hiding this comment

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

LGTM

@SunRunAway SunRunAway added status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. needs-cherry-pick-4.0 and removed status/LGT1 Indicates that a PR has LGTM 1. labels Apr 1, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Apr 1, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Apr 1, 2020

@sduzh merge failed.

@sduzh
Copy link
Contributor Author

sduzh commented Apr 1, 2020

Seems like the integration-test needs an update. @SunRunAway

@SunRunAway
Copy link
Contributor

Seems like the integration-test needs an update. @SunRunAway

That's right, I'll do it later.

@SunRunAway
Copy link
Contributor

/run-all-tests

@SunRunAway
Copy link
Contributor

/run-common-test tidb-test=pr/1049
/run-integration-common-test tidb-test=pr/1049

@codecov
Copy link

codecov bot commented Apr 29, 2020

Codecov Report

Merging #15934 into master will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master     #15934   +/-   ##
===========================================
  Coverage   79.9868%   79.9868%           
===========================================
  Files           509        509           
  Lines        138679     138679           
===========================================
  Hits         110925     110925           
  Misses        18798      18798           
  Partials       8956       8956           

@SunRunAway

This comment has been minimized.

@SunRunAway
Copy link
Contributor

@wshwsh12 @zz-jason PTAL again. Thanks

@zz-jason
Copy link
Member

/merge

@sre-bot
Copy link
Contributor

sre-bot commented Apr 30, 2020

Your auto merge job has been accepted, waiting for:

  • 15274

@sre-bot
Copy link
Contributor

sre-bot commented Apr 30, 2020

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Apr 30, 2020

@sduzh merge failed.

@zz-jason
Copy link
Member

/run-common-test tidb-test=pr/1049
/run-integration-common-test tidb-test=pr/1049

@sre-bot
Copy link
Contributor

sre-bot commented May 6, 2020

cherry pick to release-3.0 in PR #16976

@sre-bot
Copy link
Contributor

sre-bot commented May 6, 2020

cherry pick to release-3.1 in PR #16977

@sre-bot
Copy link
Contributor

sre-bot commented May 6, 2020

cherry pick to release-4.0 in PR #16978

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/expression contribution This PR is from a community contributor. status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The 'xor' result is different between tidb and mysql8
5 participants