[ 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