[ 
https://issues.apache.org/jira/browse/CALCITE-7531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18081332#comment-18081332
 ] 

Julian Hyde commented on CALCITE-7531:
--------------------------------------

Making {{isNullable}} final was my decision. It was the right choice, but I'm 
sorry you got burned.

> Add to BasicSqlType constructor accepting precision, scale and nullability
> --------------------------------------------------------------------------
>
>                 Key: CALCITE-7531
>                 URL: https://issues.apache.org/jira/browse/CALCITE-7531
>             Project: Calcite
>          Issue Type: Task
>          Components: core
>            Reporter: Sergey Nuyanzin
>            Assignee: Sergey Nuyanzin
>            Priority: Major
>              Labels: pull-request-available
>
> The issue that before CALCITE-6552 {{isNullable}} in {{AbstractSqlType}} was 
> not {{final}}.
> In Flink we set it after constructor invocation.
> After that it became final and now the same code doesn't work.
> At the same time {{BasicSqlType}} doesn't allow to pass precision, scale and 
> nullability in one constructor.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to