executor: add JSON opaque value condition to everywhere

[YangKeao]

Signed-off-by: YangKeao [email protected]

What problem does this PR solve?

Issue Number: close #37387

Problem Summary:

Cast the json opaque value to other types will cause ERROR 1105 (HY000): Unknown type code in JSON

What is changed and how it works?

Add the JSON opaque value to the places where we haven't exhausted all TypeCode.

The tikv implementation of json opaque doesn't have this issue, as it cannot be compiled if I didn't exhaust every condition.

Check List

Tests

• Unit test
• Integration test
• Manual test (add detailed scripts or steps below)
• No code

select cast(json_extract(json_objectagg('a', b'010101'), '$.a') as SIGNED);

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.

Fix the issue that the json opaque value cannot be casted to other types.

[ti-chi-bot]

[REVIEW NOTIFICATION]

This pull request has been approved by:

• xhebox
• xiongjiwei

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

[sre-bot]

Code Coverage Details: https://codecov.io/github/pingcap/tidb/commit/4ef62949401c365f10b4220b26f5f15a14f0da34

[xhebox]

Can we add the test for select cast(json_extract(json_objectagg('a', b'010101'), '$.a') as SIGNED);?

[xiongjiwei]

you can add the manual test into expression/integration_test.go, like the others, or some other place.

[YangKeao]

you can add the manual test into expression/integration_test.go, like the others, or some other place.
Can we add the test for select cast(json_extract(json_objectagg('a', b'010101'), '$.a') as SIGNED);?

Good! I added a test for complicated cast(json_extract(json_objectagg())) and a simpler cast(cast(b'0101' as json) as signed).

[xiongjiwei]

/merge

[ti-chi-bot]

This pull request has been accepted and is ready to merge.

Commit hash: 2603fa6

[sre-bot]

TiDB MergeCI notify

🔴 Bad News! [2] CI still failing after this pr merged.
These failed integration tests don't seem to be introduced by the current PR.

CI Name	Result	Duration	Compare with Parent commit
idc-jenkins-ci-tidb/common-test	🔴 failed 2, success 9, total 11	17 min	Existing failure
idc-jenkins-ci-tidb/integration-common-test	🔴 failed 2, success 15, total 17	15 min	Existing failure
idc-jenkins-ci/integration-cdc-test	🟢 all 36 tests passed	31 min	Existing passed
idc-jenkins-ci-tidb/tics-test	🟢 all 1 tests passed	7 min 5 sec	Existing passed
idc-jenkins-ci-tidb/integration-ddl-test	🟢 all 6 tests passed	6 min 56 sec	Existing passed
idc-jenkins-ci-tidb/sqllogic-test-2	🟢 all 28 tests passed	5 min 24 sec	Existing passed
idc-jenkins-ci-tidb/sqllogic-test-1	🟢 all 26 tests passed	4 min 33 sec	Existing passed
idc-jenkins-ci-tidb/mybatis-test	🟢 all 1 tests passed	3 min 16 sec	Existing passed
idc-jenkins-ci-tidb/integration-compatibility-test	🟢 all 1 tests passed	2 min 35 sec	Existing passed
idc-jenkins-ci-tidb/plugin-test	🟢 build success, plugin test success	4min	Existing passed