[CALCITE-5574] Break MockCatalogReaderSimple#init into smaller methods

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.35.0
• Component/s: tests
• Labels: pull-request-available
• Assignee: Dan Zou
• Reporter: Benchao Li
• Created: 12/Mar/23 03:46
• Updated: 27/Jul/23 10:22
• Resolved: 18/Mar/23 07:59

Description

In the discussion of https://github.com/apache/calcite/pull/3092, MockCatalogReaderSimple#init is too long to add new code, we should break it into smaller ones. One possible way is to move each table registration to a separate one method.

Issue Links

• is related to: CALCITE-5553 RelStructuredTypeFlattener produces bad plan for single field struct
• links to: GitHub Pull Request #3111