[IGNITE-23650] Calcite engine. Refactor IndexScan/TableScan classes hierarchy

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.17
• Labels: calcite2-required ise
• Assignee: Aleksey Plekhanov
• Reporter: Aleksey Plekhanov
• Created: 11/Nov/24 17:07
• Updated: 21/Nov/24 06:39
• Resolved: 21/Nov/24 06:39

Description

We have duplicated code related to partition reservation for IndexScan/TableScan classes. It would be better to have common parent class (for example, AbstractCacheScan) for both of these classes. But IndexScan is inherited from AbstractIndexScan, and this class also used by RuntimeSortedIndex, so AbstractIndexScan can be inherited from AbstractCacheScan. RuntimeSortedIndex redefine most of the methods of AbstractIndexScan as no-op, and it would be better to use some kind of tree iterator in RuntimeSortedIndex instead of AbstractIndexScan.
This refactoring is also required for IGNITE-23562 (the new node IndexCountScan can be a child of AbstractCacheScan).

Issue Links

• is depended upon by: IGNITE-23562 Calcite engine. Partitions are not reserved for IgniteIndexCount operator
• links to: GitHub Pull Request #11650