[ https://issues.apache.org/jira/browse/CALCITE-5908?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Julian Hyde reopened CALCITE-5908: ---------------------------------- > Remove unnecessary 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 > Labels: pull-request-available > > 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)