[jira] Subscription: PIG patch available

2013-12-01 Thread jira
Issue Subscription Filter: PIG patch available (8 issues) Subscriber: pigdaily Key Summary PIG-3592Should not try to create success file for non-fs schemes like hbase https://issues.apache.org/jira/browse/PIG-3592 PIG-3587add functionality for rolling over dates

[jira] [Comment Edited] (PIG-3585) Make union work with tez

2013-12-01 Thread Cheolsoo Park (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3585?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13836138#comment-13836138 ] Cheolsoo Park edited comment on PIG-3585 at 12/1/13 11:13 PM: --

[jira] [Updated] (PIG-3585) Make union work with tez

2013-12-01 Thread Cheolsoo Park (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3585?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Cheolsoo Park updated PIG-3585: --- Attachment: PIG-3585-2.patch One minor clean-up. I made getPackage() take multiple inputs as parameter

[jira] [Updated] (PIG-3585) Make union work with tez

2013-12-01 Thread Cheolsoo Park (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3585?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Cheolsoo Park updated PIG-3585: --- Resolution: Fixed Status: Resolved (was: Patch Available) Committed to tez branch. > Make uni

Re: Review Request 15931: PIG-3585 Implement union in Tez

2013-12-01 Thread Cheolsoo Park
> On Dec. 1, 2013, 5:06 p.m., Rohini Palaniswamy wrote: > > The code is fine if we have union after some processing. But for simple > > load and union case as below, this will create 3 vertices - 2 load vertices > > and one union vertex. > > > > a = load 'a' > > b = load 'b' > > c = union a,

[jira] [Commented] (PIG-3585) Make union work with tez

2013-12-01 Thread Cheolsoo Park (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3585?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13836119#comment-13836119 ] Cheolsoo Park commented on PIG-3585: Yes, it should be possible. Alternatively, I am thi

Re: Review Request 15931: PIG-3585 Implement union in Tez

2013-12-01 Thread Rohini Palaniswamy
> On Dec. 1, 2013, 5:06 p.m., Rohini Palaniswamy wrote: > > The code is fine if we have union after some processing. But for simple > > load and union case as below, this will create 3 vertices - 2 load vertices > > and one union vertex. > > > > a = load 'a' > > b = load 'b' > > c = union a,

[jira] [Commented] (PIG-3585) Make union work with tez

2013-12-01 Thread Rohini Palaniswamy (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3585?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13836074#comment-13836074 ] Rohini Palaniswamy commented on PIG-3585: - +1. The code is good if we have union aft

Re: Review Request 15931: PIG-3585 Implement union in Tez

2013-12-01 Thread Cheolsoo Park
> On Dec. 1, 2013, 5:06 p.m., Rohini Palaniswamy wrote: > > The code is fine if we have union after some processing. But for simple > > load and union case as below, this will create 3 vertices - 2 load vertices > > and one union vertex. > > > > a = load 'a' > > b = load 'b' > > c = union a,

Re: Review Request 15931: PIG-3585 Implement union in Tez

2013-12-01 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15931/#review29567 --- Ship it! The code is fine if we have union after some processing. B