planner: fix wrong TableDual plans caused by comparing Binary and Bytes incorrectly (#23860)

[ti-srebot]

cherry-pick #23860 to release-5.0
You can switch your code base to this Pull Request by using git-extras:

# In tidb repo:
git pr https://github.com/pingcap/tidb/pull/23918

After apply modifications, you can push your change to this PR via:

git push git@github.com:ti-srebot/tidb.git pr/23918:release-5.0-af0982805940

---

What problem does this PR solve?

Issue Number: close #23846

Problem Summary: planner: fix wrong TableDual plans caused by comparing Binary and Bytes incorrectly

What is changed and how it works?

planner: fix wrong TableDual plans caused by comparing Binary and Bytes incorrectly

Check List

Tests

• Unit test

Release note

• planner: fix wrong TableDual plans caused by comparing Binary and Bytes incorrectly

[ti-srebot]

/run-all-tests

[ti-srebot]

@qw4990 you're already a collaborator in bot's repo.

[eurekaka]

@qw4990 please resolve conflicts.

[eurekaka]

/lgtm

[qw4990]

PTAL @time-and-fate @rebelice

[qw4990]

/run-all-tests

[qw4990]

/run-all-tests

[time-and-fate]

Seems we need to wait #24041 merged to get this PR pass TestIssue23846().

[zimulala]

@qw4990
The test failed：

[2021-04-16T14:56:13.043Z] FAIL: integration_test.go:3137: testIntegrationSuite.TestIssue23846
[2021-04-16T14:56:13.043Z]
[2021-04-16T14:56:13.043Z] integration_test.go:3143:
[2021-04-16T14:56:13.043Z]     tk.MustQuery("select count(*) from t where a=0x00A4EEF4FA55D6706ED5").Check(testkit.Rows("1"))
[2021-04-16T14:56:13.043Z] /home/jenkins/agent/workspace/tidb_ghpr_unit_test/go/src/github.com/pingcap/tidb/util/testkit/testkit.go:63:
[2021-04-16T14:56:13.043Z]     res.c.Assert(resBuff.String(), check.Equals, needBuff.String(), res.comment)
[2021-04-16T14:56:13.043Z] ... obtained string = "[0]\n"
[2021-04-16T14:56:13.043Z] ... expected string = "[1]\n"
[2021-04-16T14:56:13.043Z] ... sql:select count(*) from t where a=0x00A4EEF4FA55D6706ED5, args:[]

[time-and-fate]

/run-all-tests

[zimulala]

/merge

[ti-chi-bot]

@zimulala: /merge is only allowed for the committers in list.

In response to this:

/merge

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository.

[ti-chi-bot]

@ti-srebot: Your PR was out of date, I have automatically updated it for you.

At the same time I will also trigger all tests for you:

/run-all-tests

If the CI test fails, you just re-trigger the test that failed and the bot will merge the PR for you after the CI passes.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository.