You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Nikolay Izhikov (Jira)" <ji...@apache.org> on 2021/10/08 13:10:00 UTC
[jira] [Commented] (FLINK-24389) getDescription() in
`CatalogTableImpl` should check null
[ https://issues.apache.org/jira/browse/FLINK-24389?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17426173#comment-17426173 ]
Nikolay Izhikov commented on FLINK-24389:
-----------------------------------------
Patch [1] for the issue is ready.
Please, review.
[1] https://github.com/apache/flink/pull/17438
> getDescription() in `CatalogTableImpl` should check null
> --------------------------------------------------------
>
> Key: FLINK-24389
> URL: https://issues.apache.org/jira/browse/FLINK-24389
> Project: Flink
> Issue Type: Bug
> Components: Table SQL / API
> Reporter: Neng Lu
> Assignee: Nikolay Izhikov
> Priority: Major
> Labels: pull-request-available, starter
>
> ```
> @Override
> public Optional<String> getDescription() {
> return Optional.of(getComment());
> }
> ```
> If the table comment is not set, then `getDescription` will throw NullPointerException
> [https://github.com/apache/flink/blame/5b9e7882207357120717966d8bf7efd53c53ede5/flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/catalog/CatalogTableImpl.java#L69]
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)