planner: check index valid while forUpdateRead (#22152)

[ti-srebot]

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

# In tidb repo:
git pr 22365

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

git push git@github.com:ti-srebot/tidb.git pr/22365:ti-srebot:release-5.0-rc-4088c2011ba7

---

What problem does this PR solve?

cherry pick #21596 to master as a temporary solution.

Issue Number: close #21498

Problem Summary:

ForUpdateRead will read data using schema version of startTS, this may lead to incorrectness, the cases are listed in #21498

What is changed and how it works?

What's Changed:

For RC and forUpdateRead in RR, check if index's state is still StatePublic.

How it Works:

This check confirms index between snapshotTS and forUpdateTS is not changed, otherwise, the index should not be used.

Related changes

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

Check List

Tests

• Unit test

Side effects

• Performance regression
  • Consumes more CPU, there is a little regression testing with tpc-c

Release note

• Fix a bug that schema change will lead to incorrectness when forUpdateRead

[ti-srebot]

/run-all-tests

[ti-srebot]

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

[cfzjywxk]

5.0-rc will release today maybe we don't need to pick it to release-5.0-rc again.

[jackysp]

5.0-rc will release today maybe we don't need to pick it to release-5.0-rc again.

yes