Dawid Wysakowicz created FLINK-14296:
----------------------------------------
Summary: SqlNodes in the parser module should use an Optional for
optional parameters
Key: FLINK-14296
URL: https://issues.apache.org/jira/browse/FLINK-14296
Project: Flink
Issue Type: Improvement
Components: Table SQL / API
Reporter: Dawid Wysakowicz
I want to suggest using Optional for optional parameters in classes such as
SqlCreateTable/SqlCreateView/SqlTableColumn etc.
Right now we must check against null at different locations e.g.:
{code}
SqlNodeList partitionKey = sqlCreateTable.getPartitionKeyList();
if (partitionKey != null) {
partitionKeys = partitionKey
.getList()
.stream()
.map(p -> ((SqlIdentifier) p).getSimple())
.collect(Collectors.toList());
}
{code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
