executor: fix build table reader for index join on partition table

[tiancaiamao]

What problem does this PR solve?

Issue Number: close #19145

Problem Summary:

In IndexLookUpMergeJoin, the inner join table reader executor is built at runtime.
The inner table may be a partition table, but the old logic dataReaderBuilder.buildTableReaderForIndexJoin does not handle that.

What is changed and how it works?

What's Changed:
How it Works:

• Turn the executor TableReader into PartitionTable->TableReader in buildTableReaderForIndexJoin
• Key range calculating logic is moved from dataReaderBuilder to Executor, because table ID is unknown at databReaderBuilder

This fix works on the master branch, 4.0 is fixed by #19151

Check List

Tests

• Unit test

Release note

• No release note

[tiancaiamao]

PTAL @lzmhhh123 @lysu @imtbkcat

[lzmhhh123]

Does IndexReader have the same problem?

[codecov]

Codecov Report

Merging #19210 into master will decrease coverage by 0.0223%.
The diff coverage is 61.2903%.

@@               Coverage Diff                @@
##             master     #19210        +/-   ##
================================================
- Coverage   79.0998%   79.0774%   -0.0224%     
================================================
  Files           549        549                
  Lines        149678     149236       -442     
================================================
- Hits         118395     118012       -383     
+ Misses        21761      21690        -71     
- Partials       9522       9534        +12     

[tiancaiamao]

Does IndexReader have the same problem?

IndexReader have already handled the partition table as join inner table case.

But not all the cases are covered until now. As I've said on another PR #18862

In buildExecutorForIndexJoinInternal, there are some other cases, I'm not sure when the code run there:

switch v := plan.(type) {
case *plannercore.PhysicalTableReader:
case *plannercore.PhysicalIndexReader:
case *plannercore.PhysicalIndexLookUpReader:
case *plannercore.PhysicalUnionScan:
case *plannercore.PhysicalProjection:
case *plannercore.PhysicalSelection:
case *mockPhysicalIndexReader:
}

• case *plannercore.PhysicalTableReader:
• case *plannercore.PhysicalIndexReader:
• case *plannercore.PhysicalIndexLookUpReader:
• case *plannercore.PhysicalUnionScan:
• case *plannercore.PhysicalProjection:
• case *plannercore.PhysicalSelection:
• case *mockPhysicalIndexReader:

I'll handle the remain cases later, after I solve this bug #19089

@lzmhhh123

[lzmhhh123]

LGTM

[lysu]

/approve

[lysu]

/merge

[ti-srebot]

/run-all-tests

[ti-srebot]

@tiancaiamao merge failed.

[zz-jason]

/merge

[ti-srebot]

Your auto merge job has been accepted, waiting for:

• 19266
• 19248

[ti-srebot]

/run-all-tests

[ti-srebot]

@tiancaiamao merge failed.

[tiancaiamao]

/run-integration-copr-test