Re: Uploading patches for review

2013-06-06 Thread Cheolsoo Park
I agree that we should use the RB more often. On Wed, Jun 5, 2013 at 6:50 PM, Jarek Jarcec Cecho jar...@apache.orgwrote: I can definitely recommend using the review board. We are using it already in all other projects that I'm participating and it's really helping us out. Jarcec On Wed,

Re: Uploading patches for review

2013-06-06 Thread Julien Le Dem
+1 On Wed, Jun 5, 2013 at 11:53 PM, Cheolsoo Park piaozhe...@gmail.com wrote: I agree that we should use the RB more often. On Wed, Jun 5, 2013 at 6:50 PM, Jarek Jarcec Cecho jar...@apache.orgwrote: I can definitely recommend using the review board. We are using it already in all other

[jira] [Commented] (PIG-3346) New property that controls the number of combined splits

2013-06-06 Thread Cheolsoo Park (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3346?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13677298#comment-13677298 ] Cheolsoo Park commented on PIG-3346: [~aniket486], you're absolutely right. Like the

[jira] [Commented] (PIG-3346) New property that controls the number of combined splits

2013-06-06 Thread Aniket Mokashi (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3346?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13677361#comment-13677361 ] Aniket Mokashi commented on PIG-3346: - Sounds good. Please feel free to post a review

[jira] [Created] (PIG-3352) Bzip2TextInputFormat can duplicate records across splits

2013-06-06 Thread Jason Lowe (JIRA)
Jason Lowe created PIG-3352: --- Summary: Bzip2TextInputFormat can duplicate records across splits Key: PIG-3352 URL: https://issues.apache.org/jira/browse/PIG-3352 Project: Pig Issue Type: Bug

[jira] [Updated] (PIG-3352) Bzip2TextInputFormat can duplicate records across splits

2013-06-06 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3352?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Lowe updated PIG-3352: Attachment: blockEndingInRecordWithCR.txt.bz2 Discovered this while investigating HADOOP-9622. Attaching a

[jira] Subscription: PIG patch available

2013-06-06 Thread jira
Issue Subscription Filter: PIG patch available (17 issues) Subscriber: pigdaily Key Summary PIG-3346New property that controls the number of combined splits https://issues.apache.org/jira/browse/PIG-3346 PIG-Fix remaining Windows core unit test failures

Re: Uploading patches for review

2013-06-06 Thread Alan Gates
I think it's fine for a reviewer to ask for a particular patch to be put in review board. I think it would also be fine to put in our HowToContribute doc that for larger patches putting it in review board may help get it reviewed more quickly. I'm not in favor of requiring it, as some

I need a small help regaridng passing param_file to pig script

2013-06-06 Thread Deepak Kumar Dabi
Hi, I am pretty sure that I have try each and every oprtion to give parament from cmd line but none of them is working in pig can anybody please help.I just want to pass a param file to pig scrip. $ pig -x local -m input=/home/hduser/deepak/pig/tabal.param /home/hduser/deepak/pig/script.pig

[jira] [Commented] (PIG-3250) Pig dryrun generates wrong output in .expanded file for 'SPLIT....OTHERWISE...' command

2013-06-06 Thread Xuefu Zhang (JIRA)
[ https://issues.apache.org/jira/browse/PIG-3250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13677811#comment-13677811 ] Xuefu Zhang commented on PIG-3250: -- +1 Thanks for the patch, Johnny. Pig