[
https://issues.apache.org/jira/browse/CALCITE-5908?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17752102#comment-17752102
]
Julian Hyde commented on CALCITE-5908:
--------------------------------------
Do you understand why those checks were added? Read the file history.
> Remove misleading NULL checks in CalciteSchema
> ----------------------------------------------
>
> Key: CALCITE-5908
> URL: https://issues.apache.org/jira/browse/CALCITE-5908
> Project: Calcite
> Issue Type: Improvement
> Components: core
> Affects Versions: 1.35.0
> Reporter: Ran Tao
> Assignee: Ran Tao
> Priority: Minor
>
> When i read the source code of `CalciteSchema` found so many null checks
> below:
>
> {code:java}
> if (tableMap == null) {
> this.tableMap = new NameMap<>();
> } else {
> this.tableMap = Objects.requireNonNull(tableMap, "tableMap");
> }
> if (latticeMap == null) {
> this.latticeMap = new NameMap<>();
> } else {
> this.latticeMap = Objects.requireNonNull(latticeMap, "latticeMap");
> } {code}
> i think we can remove it. it was misleading.
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
