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

Sushanth Sowmyan commented on HCATALOG-60:
------------------------------------------

(Updating to let you know that I'm looking through this - In the meanwhile, 
could you please also create a reviewboard submission for this? Makes community 
reviews easier : https://reviews.apache.org/ ( You can use 
https://reviews.apache.org/r/1197/ as a template ) )

> HCatBaseOutputCommitter.abortJob and HCatBaseOutputCommitter.commitJob 
> assumes a getLocation() always returns path
> ------------------------------------------------------------------------------------------------------------------
>
>                 Key: HCATALOG-60
>                 URL: https://issues.apache.org/jira/browse/HCATALOG-60
>             Project: HCatalog
>          Issue Type: Bug
>    Affects Versions: 0.3
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>              Labels: hbase, refactoring
>         Attachments: HCatalog-60.patch, HCatalog-60_rev2.patch
>
>
> Some table sources have no need for paths (ie HBase) and thus 
> JobInfo.location may be null. Looks like commitJob was trying to do a test 
> before acting on the location but buggy. abortJob does not have such check.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to