+1
Verified keys
Checked LICENSE, README, RELEASE_NOTES, CHANGES files, rat report.
Built the source
Tried running queries both using local mode and cluster
Two minor issues, that doesn’t need to block this RC
1. I think we should update README to indicate the choice of execution engine.
2. pig —
+1
On Thu, Sep 18, 2014 at 5:50 PM, Mona Chitnis wrote:
>
> +1 (non-binding)
> Mona Chitnis
> Yahoo!
>
> On Thursday, September 18, 2014 8:48 AM, Ashutosh Chauhan
> wrote:
>
>
> +1
>
> On Wed, Sep 17, 2014 at 7:02 PM, Daniel Dai wrote:
>
>> +1
>>
>> On Wed, Sep 17, 2014 at 11:12 AM, Pra
---
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
There is no processedOperators.add happening. Is this variable needed ?
trunk/src/org/apache/pig/newplan/logical/rules/ConstantCalculator.java
<https://reviews.apache.org/r/24789/#comment89755>
does it make sense to do this setPlan in moveTree call itself?
- Thejas Nair
On Aug. 19, 2014, 5:41
Thanks Prashant!
On Tue, Apr 15, 2014 at 10:58 AM, Cheolsoo Park wrote:
> Thank you Prashant for your hard work!
>
>
> On Mon, Apr 14, 2014 at 5:37 PM, Daniel Dai wrote:
>
>> Thanks Prashant!
>>
>> On Mon, Apr 14, 2014 at 5:30 PM, Prashant Kommireddi
>> wrote:
>> > The Pig team is happy to ann
tps://cwiki.apache.org/confluence/display/PIG/HowToRelease
>> <https://cwiki.apache.org/confluence/display/PIG/HowToRelease>". When
>> userbuild the release, the version will be {next
>> version}-SNAPSHOT.Thanks,Daniel"*
>>
>>
>> On Fri, Apr 11, 2014
Here is my late +1.
Checked the md5 and asc keys.
Checked release notes, CHANGES.txt.
Build from source tar, tried some local queries. Checked output of
version command (pig -version)
The output of version command in binary is accurate.
However, in case of source tar, when I build using just "ant"
Congrats Rohini and Cheolsoo!
On Thu, Sep 12, 2013 at 11:24 AM, Bill Graham wrote:
> Congrats guys! Well deserved indeed.
>
>
> On Wed, Sep 11, 2013 at 10:58 PM, Jarek Jarcec Cecho wrote:
>
>> Congratulations Rohini and Cheolsoo, awesome work!
>>
>> Jarcec
>>
>> On Wed, Sep 11, 2013 at 04:24:21PM
Congrats Koji! Very well deserved!
On Wed, Sep 11, 2013 at 9:49 AM, Daniel Dai wrote:
> Congratulation! You are well deserved.
>
>
>
>
> On Wed, Sep 11, 2013 at 6:33 AM, Miguel Angel Martin junquera <
> mianmarjun.mailingl...@gmail.com> wrote:
>
>> Congratulations K
>>
>>
>> Miguel Angel Ma
+1
Verified md5 checksums of src and binary tar.gz .
Build the src tar.gz and ran queries against a hadoop 1.1 cluster, ran
fs and sh commands.
-Thejas
On 1/3/13 12:11 PM, Rohini Palaniswamy wrote:
+1. Downloaded the tar binary, checked signature, ran unit tests, piggybank
unit tests, checke
/pig/trunk/test/org/apache/pig/test/TestTypeCheckingValidatorNewLP.java
1373741
http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/test/utils/GenRandomData.java
1373741
Diff: https://reviews.apache.org/r/5414/diff/
Testing
---
Thanks,
Thejas Nair
/repos/asf/pig/trunk/test/org/apache/pig/test/TestTypeCheckingValidatorNewLP.java
1371785
http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/test/utils/GenRandomData.java
1371785
Diff: https://reviews.apache.org/r/5414/diff/
Testing
---
Thanks,
Thejas Nair
Yes, illustrate is still supported. We made lot of improvements in 0.9
in getting it working under more conditions.
Can you forward Allan's question to the list ?
Thanks,
Thejas
On 8/2/12 6:52 AM, Gianmarco De Francisci Morales wrote:
Hi,
The GSoC project that Allan is working on and that
We certainly need to look at ways to reduce the runtime of the 'unit'
tests. Some of them should be migrated to the e2e tests.
But what you want for being able to re-test easily seems to be a way to
specify specific test case within a Test*.java file . I wonder if junit
lets you do that.
-Th
heck for DATETIME should be not added here.
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/pen/AugmentBaseDataVisitor.java
<https://reviews.apache.org/r/5414/#comment19283>
check for DATETIME should be not added here.
- Thejas Nair
On July 10, 2012, 5:41 p.m., T
/repos/asf/pig/trunk/test/org/apache/pig/test/TestConversions.java
1359212
http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/test/TestPOCast.java
1359212
Diff: https://reviews.apache.org/r/5414/diff/
Testing
---
Thanks,
Thejas Nair
Earlier implementation of illustrate used the pig local mode execution
engine (which corresponds to the time when paper was published) .
As part of illustrate reword in PIG-1712, Yan replaced the default Map
and Reduce context objects with a IllustratorContext. Look for
IllustratorContext and
That certainly looks like a bug. The replicated join should not flatten
the tuple.
I didn't actually know that pig supported doing joins on tuples (i guess
it does not allow that on maps and bags).
-Thejas
On 6/21/12 11:29 AM, Jonathan Coveney wrote:
Am posting before making a ticket just to
bcc'ing the user list.
1. Order-by
The comparison against hive order-by is misleading. Hive does not do
total ordering, unless you use a single reducer.
But yes, in case of pig, the sampling phase is unnecessary, if you use a
single reducer. A single reducer can make sense if the data you are
1348191
Diff: https://reviews.apache.org/r/5414/diff/
Testing
---
Thanks,
Thejas Nair
In my opinion, we should only commit changes to released branches that
are either critical bug fixes, or very useful minor changes which are
not likely to affect the stability of the branch.
This change would fall into 2nd category.
Thanks,
Thejas
On 4/26/12 2:32 PM, Bill Graham wrote:
What
ection pushdown to work through
Split operators.
D
On Wed, Apr 25, 2012 at 12:52 PM, Thejas Nair
mailto:the...@hortonworks.com>> wrote:
> cc'ing dev@pig as this is a pig issue.
>
> Aniket, What you saw is not related to PIG-2339 .
>
> In
rs separately to the loader. But, that
doesn't work in pig. (I can open a jira, but I havent done enough
investigation at the code level). Thoughts?
Thanks,
Aniket
On Tue, Apr 24, 2012 at 7:00 PM, Thejas Nair mailto:the...@hortonworks.com>> wrote:
The issue was not speci
+1 .
Checked checksum and signatures of all 3 packages. Ran simple queries in
MR and local modes using tar package on unsecure cluster, and rpm
package on secure cluster.
Thanks,
Thejas
On 4/20/12 12:39 AM, Daniel Dai wrote:
Hi,
I have created a candidate build for Pig 0.10.0.
Keys used to
On 4/18/12 3:24 PM, Russell Jurney wrote:
I'm in. I'm going to work on getting piggybank on github, including for Jython
and JRuby UDFs.
I think the major work involved there is to figure out how to lower the
barrier to contribute and having independent release cycles for the udfs
while al
Count me in
-Thejas
On 4/18/12 2:18 PM, Dmitriy Ryaboy wrote:
Hi folks,
The Analytics Infra team at Twitter will be hosting a Pig hackday on May 11.
On the agenda:
- get newcomers set up with the apache ticket process
- review and commit a bunch of stuff that's not been getting love
- hack on
fyi, For those who expressed interest in mentoring students for GSoC, seem
email below for instructions to register .
Here is the apache mentoring guide -
http://community.apache.org/guide-to-being-a-mentor.html
- Forwarded Message -
From: Ulrich Stärk
To: p...@apache.org; code-awa
Sounds good to me.
My thoughts on the costs of this change -
- svn will still retain the history of the moved files. So that is not a
problem.
- build.xml would need some minor changes
- some extra steps will be required to apply the patches generated
against old directory structure.
Thanks,
See initial sections in
http://infolab.stanford.edu/~olston/publications/vldb09.pdf for overview
of logical plan.
LogicalPlanGenerator.g is a the place where logical plan is created from
parse tree. You would need to look at antlr basics to understand that.
(almost?) all pig relational opera
+1
Checked the md5 checksums, keys of all 3 packages. Ran some simple
queries using the rpm package on a secure and unsecure cluster. Checked
the -version command.
-Thejas
On 1/18/12 11:21 AM, Daniel Dai wrote:
For your information, I took a shortcut last night to refresh the candidate
1 to
This is the 2nd apache user group that has reported that emails to my
@hortonworks.com address are bouncing.
Is anybody else seeing this ? Any way to fix it ?
I tried searching for a solution for this, but didn't find any.
-Thejas
Original Message
Subject: warning from u...
t 0.10 stabilize (via a few dot releases), then move all versions
up in one fell swoop -- so 0.10 line becomes 1.0, 0.11 becomes 1.1, and if at
that point 0.12 also exists, it becomes 1.2.
D
On Mon, Oct 24, 2011 at 6:10 PM, Thejas Nair wrote:
Dmitriy,
I think what you are saying is something
I can find two issues reported by Jonathan, both related to the parser.
One having to do with nested statement syntax, and other to do with
speed of parsing. Both of these should get looked into before next release.
But I don't see a whole lot issues with the parser changes or 0.9 in
general,
- Avro (1.5.4, 1.4.1, ...)
- Cassandra (1.0.0, 0.8.7, ...)
- Chukwa (0.4.0, 0.3.0, ...)
- Hama (0.3.0, 0.2.0, ...)
- Hbase (0.90.4, 0.90.3, 0.90.2, 0.90.1, ...)
- Hive (Releases - 0.7.1, 0.7.0, 0.6.0, ...)
- Zookeeper (3.3.3, 3.3.2, 3.2.2, 3.1.2, ...)
S
On 10/24/11 12:43 PM, Dmitriy Ryaboy wrote:
We are finding a fair number of issues trying to move from Pig 0.8.1 to 0.9,
and I don't think those issues are fixed in 10, either.. not sure that this
"stabilization" process has happened yet.
D
What kind of issues are these ? Are they related to
I see that you are running in local mode. You might want to specify a
-Xmx value if aren't doing that, the default max heap size is usually
quite low.
Is the udf that you are using holding onto lot of data ? Does this
happen with other builtin udfs as well ?
-Thejas
On 10/17/11 1:19 AM, J
Sending this email for getting wider attention.
I propose disabling LogicalExpressionSimplifier optimizer rule, because
the complexity of that rule and number of bugs that seem to come from
there does not justify the expected performance gains -
https://issues.apache.org/jira/browse/PIG-2316?fo
I would vote for option C - i would like the user to sign off in each
place the feature is used.
pig scripts will be modified over time, and person making the edit might
not notice that the checks are turned off elsewhere in the script. If it
is set in a properties file, it could get inadverte
is is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/1929/
> ---
>
> (Updated 2011-09-16 18:11:08)
>
>
> Review request for pig and Thejas Nair.
>
>
> Summary
> ---
&
/physicalLayer/relationalOperators/POPartialAgg.java
<https://reviews.apache.org/r/1817/#comment4397>
removed the extra ";" in the patch checked in.
- Thejas
On 2011-09-15 17:27:08, Thejas Nair wrote:
>
> ---
>
------
On 2011-09-15 17:27:08, Thejas Nair wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/1817/
> ---
>
PRECS_TO_SAMPLE_SZ_REDUCTION?
yes, fixing the comment.
- Thejas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1817/#review1894
-----
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1817/
---
(Updated 2011-09-15 17:27:08.965065)
Review request for pig, Daniel Dai and Dmit
gg.java
<https://reviews.apache.org/r/1817/#comment4334>
the cast here is actually unnecessary, removing it.
- Thejas
On 2011-09-12 23:55:12, Thejas Nair wrote:
>
> ---
> This is an automatically generated e-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1817/
---
Review request for pig, Daniel Dai and Dmitriy Ryaboy.
Summary
---
See PIG-
FYI-
Yourkit is very useful java profiling tool and they give license for
free for use by open source projects.
I am planning to request license for use by pig committers.
But they need a reference from the web pages of the project to their
website. - http://www.yourkit.com/purchase/index.jsp
:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/1627/
> ---
>
> (Updated 2011-08-23 17:08:10)
>
>
> Rev
:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/1600/
> ---
>
> (Updated 2011-08-19 20:36:09)
>
>
> Rev
Dmitriy,
You don't realize how lucky you are! ;)
I have been trying hard to reproduce this problem, so that I can check
if the patch in PIG-2055 actually fixes the issue. I ran build+
(small)test in a loop for 2000+ times, and this hasn't happened yet.
If this is happening (almost) consistentl
Dear pig community,
Please welcome Gianmarco as the newest committer to apache pig project!
He has been contributing to pig for more than a year. His contributions
include the use of binary comparator in secondary sort , support for
default output in split operator, use of scalar expression in
+1
Ran queries in local mode on mac, test-commit, and verified md5 checksum.
-Thejas
On 7/22/11 4:24 PM, Alan Gates wrote:
+1.
Ran the test-commit, tutorial, and quick sanity test against a real cluster on
Linux, ran a quick sanity test in local mode on Mac. Checked signature key and
md5.
A
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/767/#review1105
---
Ship it!
+1
- thejas
On 2011-05-19 22:26:01, Daniel Dai wrote:
>
>
there is also the
danger that the alternative script also has the same bug which results
buggy benchmark data.
-Thejas
On 7/14/11 3:51 PM, Thejas Nair wrote:
On 7/14/11 2:39 PM, Alan Gates wrote:
I have posted a proposal for changes in Pig's testing that I would
like to m
On 7/14/11 2:39 PM, Alan Gates wrote:
I have posted a proposal for changes in Pig's testing that I would like to
make. https://cwiki.apache.org/confluence/display/PIG/PigTestProposal Please
take a look and provide feedback.
Alan.
+1 for the proposal.
-Thejas
I had 3 dimensions?
I doubt if it is because of number of dimensions, I think there might
have been something else in the query that prevented the group-by's from
being combined together.
Do you still have the original script ? Can you send the script (and
maybe the explain output) ?
+1 to what Gianmarco said about the place to do it. See sample_clause
in LogicalPlanGenerator.g.
I tried the expanded query (2 dimensions) with 0.8, it results only in 2
MR jobs, the 1st MR job has all the computation being done in a single
MR job. The 2nd MR job just concats the outputs into
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/763/#review687
---
Ship it!
+1
- thejas
On 2011-05-19 17:46:48, Daniel Dai wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/673/#review606
---
Ship it!
+1
- thejas
On 2011-04-28 20:56:30, Daniel Dai wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/670/#review594
---
Ship it!
+1
- thejas
On 2011-04-27 21:28:47, Daniel Dai wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/621/#review500
---
Ship it!
+1
- thejas
On 2011-04-19 00:37:31, Daniel Dai wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/624/#review499
---
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/newplan/lo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/587/#review461
---
Ship it!
+1
- thejas
On 2011-04-13 00:11:37, Daniel Dai wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/544/#review382
---
Ship it!
+1
- thejas
On 2011-04-04 18:10:55, Daniel Dai wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/524/#review375
---
Ship it!
+1
- thejas
On 2011-03-24 12:22:48, Daniel Dai wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/526/#review354
---
Ship it!
+1
- thejas
On 2011-03-25 11:51:15, Daniel Dai wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/459/#review299
---
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/newplan/lo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/459/#review295
---
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/ha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/459/#review289
---
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/newplan/lo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/459/#review282
---
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/impl/util/
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/356/#review223
---
Ship it!
- thejas
On 2011-02-14 17:00:02, Daniel Dai wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/134/#review55
---
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/builtin/Bin
71 matches
Mail list logo