planner, errno: make error code of ErrMixOfGroupFuncAndFields consistent with MySQL

[xuyifangreeneyes]

What problem does this PR solve?

Issue Number: close #22694

Problem Summary:
MySQL and TiDB report the same error about only_full_group_by check but give different error codes.

What is changed and how it works?

What's Changed & How it Works:
We remove ErrMixOfGroupFuncAndFieldsIncompatible(8123) and use ErrMixOfGroupFuncAndFields(1140) instead.

Related changes

• PR to update pingcap/docs/pingcap/docs-cn:
• Need to cherry-pick to the release branch

Check List

Tests

• Unit test
• Integration test

Release note

• make error code of ErrMixOfGroupFuncAndFields consistent with MySQL

[xuyifangreeneyes]

/run-all-tests

[ti-chi-bot]

@xuyifangreeneyes: PR needs rebase.

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 kubernetes/test-infra repository.

[xuyifangreeneyes]

/cc @winoros @AilinKid

[ti-chi-bot]

@xuyifangreeneyes: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
pull-br-integration-test	adb7015	link	true	/test pull-br-integration-test

Full PR test history. Your PR dashboard.

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 kubernetes/test-infra repository. I understand the commands that are listed here.

[ti-chi-bot]

@xuyifangreeneyes: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
pull-br-integration-test	adb7015	link	true	/test pull-br-integration-test

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 kubernetes/test-infra repository. I understand the commands that are listed here.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.1769%. Comparing base (9e3c064) to head (adb7015).
Report is 12346 commits behind head on master.

Additional details and impacted files

@@               Coverage Diff                @@
##             master     #22722        +/-   ##
================================================
+ Coverage   77.1500%   77.1769%   +0.0268%     
================================================
  Files           586        586                
  Lines        147646     147820       +174     
================================================
+ Hits         113909     114083       +174     
+ Misses        23488      23487         -1     
- Partials      10249      10250         +1     

[sre-bot]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

---

xuyifan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}