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

Ted Yu commented on HIVE-9301:
------------------------------

I stepped through similar code in debugger - the single ampersand prevents 
short circuit evaluation of the expression, leading to NPE.

> Potential null dereference in MoveTask#createTargetPath()
> ---------------------------------------------------------
>
>                 Key: HIVE-9301
>                 URL: https://issues.apache.org/jira/browse/HIVE-9301
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>         Attachments: HIVE-9301.patch
>
>
> {code}
>     if (mkDirPath != null & !fs.exists(mkDirPath)) {
> {code}
> '&&' should be used instead of single ampersand.
> If mkDirPath is null, fs.exists() would still be called - resulting in NPE.



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

Reply via email to