[
https://issues.apache.org/jira/browse/CALCITE-5574?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Benchao Li resolved CALCITE-5574.
---------------------------------
Fix Version/s: 1.35.0
Resolution: Fixed
Fixed in
https://github.com/apache/calcite/commit/ace7940731b76ac870f85a4052d9cd45767fd220
[~FrankZou] Thanks for your PR! And thanks
[~julianhyde][~snuyanzin][~asolimando] for the review.
> Break MockCatalogReaderSimple#init into smaller methods
> -------------------------------------------------------
>
> Key: CALCITE-5574
> URL: https://issues.apache.org/jira/browse/CALCITE-5574
> Project: Calcite
> Issue Type: Improvement
> Components: tests
> Reporter: Benchao Li
> Assignee: Zou
> Priority: Minor
> Labels: pull-request-available
> Fix For: 1.35.0
>
> Time Spent: 2h 50m
> Remaining Estimate: 0h
>
> In the
> [discussion|https://github.com/apache/calcite/pull/3092#discussion_r1131364712]
> 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.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
