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

Stephen Durfey commented on CRUNCH-340:
---------------------------------------

Addressed the feedback. I re-org'd the test cleanup and managed to get the 
derby.log file included in the temporary output so its auto-cleaned up at the 
end of the test suite. The custom surefire/failsafe configs are removed. 
Javadoc has been fixed, and the site.xml has been removed.

The only other change was fixing the getFieldID() method in 
TupleObjectInspector. I had it fixed correctly in CRUNCH-659, but missed it in 
the original patch here. 

> Create HCatSource and HCatTarget
> --------------------------------
>
>                 Key: CRUNCH-340
>                 URL: https://issues.apache.org/jira/browse/CRUNCH-340
>             Project: Crunch
>          Issue Type: New Feature
>            Reporter: Chao Shi
>            Assignee: Stephen Durfey
>         Attachments: 0001-CRUNCH-340-added-HCatSource-HCatTarget.patch, 
> 0001-CRUNCH-340-added-HCatSource-HCatTarget_v2.patch, CRUNCH-340.patch, 
> crunch-340-v2.patch, crunch-340-v3.patch, crunch-340-v4.patch, 
> crunch-340.patch
>
>
> This patch adds HCatSource, which enables crunch pipeline to read from Hive 
> tables. This is the very first version, leaving a few TODOs in code.
> It adds new dependency from crunch-core to hcatalog (as well as several hive 
> components). I guess maybe we should create a new subproject (e.g. 
> crunch-hcatalog) rather than add it into crunch-core.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to