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

Francis Liu commented on HCATALOG-60:
-------------------------------------

Minutes of meeting:

-For the OutputFormatContainer having HCatRecord as the Value type. Agreed as 
correct since this is what users are expected to pass and 
StorageDriver.convert() is exptected to consume.
-Description of this refactoring in a nutshell is:HCatOutputFormat, 
HCatRecordWriter, HCatOutputCommitter has been renamed to 
FileOutputFormatContainer,etc and a new thin HCatOutputFormat encapsulates the 
Container class delegating all calls to the underlying container (ie 
getRecordWriter, getOutputCommitter)
-code path is essentially identical except for some changes such as moving 
output directory validation to OutputFormat.checkOutputSpecs()

> Refactor HCatalog to support non-filebased outputformats
> --------------------------------------------------------
>
>                 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, 
> HCatalog-60_rev3.patch, HCatalog-60_rev4.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