feat(sql): code refactor for misleading-indentation(#1542)

[zzzz-vincent]

Summary about this PR

Issue Number: close #1542

Tests Check List

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

Changelog

• New Feature
• Bug Fix
• Performance Improvement
• Build/Testing/CI/CD
• Documentation
• Not for changelog (changelog entry is not required)

Documentation

• Affects user behaviors
• Contains syntax changes
• Contains variable changes
• Contains experimental features

[mergify]

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

[hustjieke]

These code causes unpredictable behavior?

[codecov]

Codecov Report

Patch coverage: 83.33% and project coverage change: +55.20 🎉

Comparison is base (a33ba87) 0.00% compared to head (bd5553f) 55.20%.

❗ Current head bd5553f differs from pull request most recent head b0ae8dc. Consider uploading reports for the commit b0ae8dc to get more accurate results

Additional details and impacted files

@@                 Coverage Diff                  @@
##           stonedb-5.7-dev    #1646       +/-   ##
====================================================
+ Coverage                 0   55.20%   +55.20%     
====================================================
  Files                    0     2031     +2031     
  Lines                    0   431427   +431427     
====================================================
+ Hits                     0   238170   +238170     
- Misses                   0   193257   +193257     

Impacted Files	Coverage Δ
sql/sql_optimizer.cc	79.24% <83.33%> (ø)

... and 2030 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[hustjieke]

These changes may cause unexpected behaviour on innodb, can you make a regression test on innodb?

The cmd is:

./mysql-test-run.pl --suite=innodb,main --mysqld=--default-storage-engine=innodb --nowarnings --force --nocheck-testcases --retry=0 --do-test-list=./suite/innodb/include/innodb_testlist --parallel=15

Thanks!

[zzzz-vincent]

@hustjieke thanks for pointing out. Will take a look soon.

[konghaiya]

From the test results, it seems that some of the previous modifications have solved the problem of primary key scanning
It should be possible to relax the restrictions on the use of primary keys,

[konghaiya]

LGTM

[RingsC]

LGTM, thanks

[mergify]

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.