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

Daniel Dai commented on PIG-3294:
---------------------------------

bq. The checking in of Hive code is ugly. We need to make sure that gets 
removed before a release so we don't end up forking.
Sure, the duplication is ugly. I already commented on the code "Will remove 
once we switch to use Hive 1.2.0". I expect to remove those classes in the next 
release. However, in this release, I don't want to create a dependency on Hive 
1.2.0. That will complicate the release process. Those classes are simple 
enough and don't likely to cause trouble for a short while.

bq. In POForEach you are visiting the physical plan at run time to determine if 
we need the last record
Yes, I can cache the flag and not do this in the backend. Will update patch 
shortly.

bq. HiveUtils.java: much of this code to convert Hive types to Pig types must 
already be in HCat. Is it not possible to re-use that?
Sure, we can consolidate these code. Since Pig don't depend on HCat, but HCat 
depends on Pig, I guess we shall rework HCat to use the same code to do the 
type conversion.

> Allow Pig use Hive UDFs
> -----------------------
>
>                 Key: PIG-3294
>                 URL: https://issues.apache.org/jira/browse/PIG-3294
>             Project: Pig
>          Issue Type: New Feature
>            Reporter: Daniel Dai
>            Assignee: Daniel Dai
>              Labels: gsoc2013, java
>             Fix For: 0.15.0
>
>         Attachments: PIG-3294-1.patch, PIG-3294-2.patch, PIG-3294-3.patch, 
> PIG-3294-4.patch, PIG-3294-before-refactory.patch
>
>
> It would be nice if Pig provide some interoperability with Hive. We can wrap 
> Hive UDF in Pig so we can use Hive UDF in Pig.
> This is a candidate project for Google summer of code 2013. More information 
> about the program can be found at 
> https://cwiki.apache.org/confluence/display/PIG/GSoc2013



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to