executor: fix some audit log error#26489
executor: fix some audit log error#26489tiancaiamao wants to merge 2 commits intopingcap:release-3.0from
Conversation
|
[REVIEW NOTIFICATION] This pull request has not been approved. To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. DetailsReviewer can indicate their review by submitting an approval review. |
|
No release note, Please follow https://github.com/pingcap/community/blob/master/contributors/release-note-checker.md |
|
@tiancaiamao: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. DetailsInstructions 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. |
|
Not for 3.X |
What problem does this PR solve?
Problem Summary:
Some of the audit log are incorrect:
https://docs.google.com/spreadsheets/d/1E59z7busbRwjb2D20QLbmAGhLT75vec44HTB5vi2WrI/edit#gid=0
What is changed and how it works?
What's Changed:
Fix the following bugs:
How it Works:
Some are caused by AST not registed in
GetStmtLabel()Some are caused by the statement execute some other SQL internally and change the statement context
Check List
Tests
Insert (".", ".",".*","") into mysql.tidb_audit_table_access before executing
Side effects
Documentation
Release note