[jira] [Commented] (KYLIN-2209) Potential NPE in StreamingController#deserializeTableDesc()

[Ted Yu (JIRA)]

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

Ted Yu commented on KYLIN-2209:
-------------------------------

Thanks Kaige for the patch.

> Potential NPE in StreamingController#deserializeTableDesc()
> -----------------------------------------------------------
>
>                 Key: KYLIN-2209
>                 URL: https://issues.apache.org/jira/browse/KYLIN-2209
>             Project: Kylin
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee:  Kaige Liu
>            Priority: Minor
>         Attachments: KYLIN-2209-fix-potential-NPE.patch
>
>
> {code}
>         TableDesc desc = null;
>         try {
>             logger.debug("Saving TableDesc " + 
> streamingRequest.getTableData());
>             desc = JsonUtil.readValue(streamingRequest.getTableData(), 
> TableDesc.class);
>         } catch (JsonParseException e) {
>             logger.error("The TableDesc definition is invalid.", e);
>             updateRequest(streamingRequest, false, e.getMessage());
> ...
>         String[] dbTable = HadoopUtil.parseHiveTableName(desc.getName());
> {code}
> There is a chance that desc is null by the time we get to 
> HadoopUtil.parseHiveTableName().
> We should check for null.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)