[
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