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

types/data: the bit data type should be treat as uint64 (#11951) #12616

Closed
wants to merge 3 commits into from
Closed

types/data: the bit data type should be treat as uint64 (#11951) #12616

wants to merge 3 commits into from

Conversation

gaoxingliang
Copy link
Contributor

What problem does this PR solve?

fix cherry-pick #11951 to release-2.1 . related issue for automatically pick is in here

@sre-bot sre-bot added the contribution This PR is from a community contributor. label Oct 11, 2019
@zz-jason zz-jason requested review from qw4990 and SunRunAway October 13, 2019 09:02
@@ -471,3 +472,21 @@ func (s *testSuite) TestPartitionInsertOnDuplicate(c *C) {
tk.MustQuery(`select * from t2`).Check(testkit.Rows("1 1"))

}

func (s *testSuite) TestBitInRange(c *C) {
Copy link
Contributor

Choose a reason for hiding this comment

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

in #11951, it is callled TestBit, why here is it called TetBItInRange

@SunRunAway SunRunAway changed the title types/data: the bit data type should be treat as uint64 types/data: the bit data type should be treat as uint64 (#11951) Oct 13, 2019
@SunRunAway
Copy link
Contributor

Should we abandon #12424 ?
And what's the difference this PR from #12424?

@gaoxingliang
Copy link
Contributor Author

Hi @SunRunAway I created this PR because the cherry pick failed because of code conflicts and that's why the method name changed. please see the PR comments and the error log it shows build error.

[2019-10-13T15:59:04.620Z] # github.com/pingcap/tidb/executor_test [github.com/pingcap/tidb/executor.test]
[2019-10-13T15:59:04.620Z] executor/insert_test.go:492:10: undefined: testSuite3
[2019-10-13T15:59:04.620Z] FAIL github.com/pingcap/tidb/executor [build failed]

@qw4990 qw4990 removed their request for review November 26, 2019 07:39
@SunRunAway
Copy link
Contributor

Hi, I've resolved the conflicts in #12424, and will close this PR.
Thank you anyway, @gaoxingliang

@SunRunAway SunRunAway closed this Nov 29, 2019
@gaoxingliang gaoxingliang deleted the fixcherrypick branch December 3, 2019 05:41
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants