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

ASF GitHub Bot commented on TAJO-1258:
--------------------------------------

Github user jinossy commented on the pull request:

    https://github.com/apache/tajo/pull/340#issuecomment-69210267
  
    +1 Looks good to me.
    I'll commit it after the TravisCI finished


> Close() for classes derived from FileAppender should be robust
> --------------------------------------------------------------
>
>                 Key: TAJO-1258
>                 URL: https://issues.apache.org/jira/browse/TAJO-1258
>             Project: Tajo
>          Issue Type: Improvement
>          Components: storage
>    Affects Versions: 0.9.0
>            Reporter: Jongyoung Park
>            Assignee: Jongyoung Park
>             Fix For: 0.10
>
>
> As I investigated, many appenders have possibility to cause NPE such as 
> TAJO-1227.
> It seems checking null for each variables is ineffective.



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

Reply via email to