Re: Creating a branch for Pig on Spark (PIG-4059)

2014-08-26 Thread Mayur Rustagi
Praveen me should be good enough for us. We will have 1-2 more folks contributing from Sigmoid but we can manage by merging patches from them. Initially it will be good to structure patches through us as we would like to avoid breaking architecture flow. Regards Mayur On Tue, Aug 26,

Re: Creating a branch for Pig on Spark (PIG-4059)

2014-08-26 Thread Jarek Jarcec Cecho
I would second Mayur - as he and Praveen are most familiar with all the changes, I would let them be the only additional branch committers. I’m expecting that they will review other contributor's patches similarly as normal committers and hence the contributors will start building Karma to get

Re: Creating a branch for Pig on Spark (PIG-4059)

2014-08-26 Thread Cheolsoo Park
Ok. I will start a vote today and get back to you in few days. On Tue, Aug 26, 2014 at 1:28 AM, Jarek Jarcec Cecho jar...@apache.org wrote: I would second Mayur - as he and Praveen are most familiar with all the changes, I would let them be the only additional branch committers. I’m

[DISCUSS] Re: Dropping support for JDK6 in Apache Hadoop

2014-08-26 Thread Rohini Palaniswamy
Pig has support for jdk7 from Pig 0.10. I think we should drop support for JDK6 from Pig 0.14 and also publish maven binaries with jdk 1.7 from Pig 0.14. Also it is high time to drop support for Hadoop 0.20. None of the other hadoop projects officially support Hadoop 0.20 anymore. I would like to

Re: [DISCUSS] Re: Dropping support for JDK6 in Apache Hadoop

2014-08-26 Thread Alan Gates
I'm +1 on both of these. But as a side note Hive actually still supports Hadoop 0.20, so you're statement below isn't quite true. Alan. Rohini Palaniswamy mailto:rohini.adi...@gmail.com August 26, 2014 at 9:36 Pig has support for jdk7 from Pig 0.10. I think we should drop support for JDK6

[jira] [Commented] (PIG-3087) Refactor TestLogicalPlanBuilder to be meaningful

2014-08-26 Thread Hao Gao (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3087?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14111354#comment-14111354 ] Hao Gao commented on PIG-3087: -- Can I try this? Refactor TestLogicalPlanBuilder to be

[jira] [Commented] (PIG-3087) Refactor TestLogicalPlanBuilder to be meaningful

2014-08-26 Thread Jonathan Coveney (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3087?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14111401#comment-14111401 ] Jonathan Coveney commented on PIG-3087: --- Please do! Refactor TestLogicalPlanBuilder

[jira] [Updated] (PIG-4141) Ship UDF/LoadFunc/StoreFunc dependent jar automatically

2014-08-26 Thread Daniel Dai (JIRA)
[ https://issues.apache.org/jira/browse/PIG-4141?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-4141: Attachment: PIG-4141-1.patch Ship UDF/LoadFunc/StoreFunc dependent jar automatically

[jira] [Created] (PIG-4141) Ship UDF/LoadFunc/StoreFunc dependent jar automatically

2014-08-26 Thread Daniel Dai (JIRA)
Daniel Dai created PIG-4141: --- Summary: Ship UDF/LoadFunc/StoreFunc dependent jar automatically Key: PIG-4141 URL: https://issues.apache.org/jira/browse/PIG-4141 Project: Pig Issue Type: New

Re: Review Request 24789: New logical optimizer rule: ConstantCalculator

2014-08-26 Thread Daniel Dai
On Aug. 25, 2014, 8:24 p.m., Thejas Nair wrote: trunk/src/org/apache/pig/newplan/logical/rules/ConstantCalculator.java, line 53 https://reviews.apache.org/r/24789/diff/2/?file=664437#file664437line53 There is no processedOperators.add happening. Is this variable needed ? Yes, we

Re: Review Request 24789: New logical optimizer rule: ConstantCalculator

2014-08-26 Thread Daniel Dai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24789/ --- (Updated Aug. 26, 2014, 10:35 p.m.) Review request for pig. Repository: pig

[jira] [Updated] (PIG-4128) New logical optimizer rule: ConstantCalculator

2014-08-26 Thread Daniel Dai (JIRA)
[ https://issues.apache.org/jira/browse/PIG-4128?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-4128: Attachment: PIG-4128-3.patch Addressing Thejas' review comments. New logical optimizer rule:

Re: Review Request 24789: New logical optimizer rule: ConstantCalculator

2014-08-26 Thread Thejas Nair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24789/#review51603 --- Ship it! Ship It! - Thejas Nair On Aug. 26, 2014, 10:35 p.m.,

[jira] [Commented] (PIG-4128) New logical optimizer rule: ConstantCalculator

2014-08-26 Thread Thejas M Nair (JIRA)
[ https://issues.apache.org/jira/browse/PIG-4128?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14111505#comment-14111505 ] Thejas M Nair commented on PIG-4128: +1 New logical optimizer rule: ConstantCalculator

[jira] [Resolved] (PIG-4128) New logical optimizer rule: ConstantCalculator

2014-08-26 Thread Daniel Dai (JIRA)
[ https://issues.apache.org/jira/browse/PIG-4128?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai resolved PIG-4128. - Resolution: Fixed Hadoop Flags: Reviewed Patch committed to trunk. Thanks Thejas for review! New

[jira] Subscription: PIG patch available

2014-08-26 Thread jira
Issue Subscription Filter: PIG patch available (15 issues) Subscriber: pigdaily Key Summary PIG-4111Make Pig compiles with avro-1.7.7 https://issues.apache.org/jira/browse/PIG-4111 PIG-4103Fix TestRegisteredJarVisibility(after PIG-4083)