[
https://issues.apache.org/jira/browse/PIG-1962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13021361#comment-13021361
]
[email protected] commented on PIG-1962:
----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/620/
-----------------------------------------------------------
Review request for pig and Richard Ding.
Summary
-------
See PIG-1962
This addresses bug PIG-1962.
https://issues.apache.org/jira/browse/PIG-1962
Diffs
-----
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/HExecutionEngine.java
1094667
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/newplan/logical/relational/LogToPhyTranslationVisitor.java
1094667
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/newplan/logical/visitor/StoreAliasSetter.java
PRE-CREATION
http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/test/TestPlanGeneration.java
PRE-CREATION
http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/test/Util.java
1094667
Diff: https://reviews.apache.org/r/620/diff
Testing
-------
Test-patch:
[exec] +1 overall.
[exec]
[exec] +1 @author. The patch does not contain any @author tags.
[exec]
[exec] +1 tests included. The patch appears to include 6 new or
modified tests.
[exec]
[exec] +1 javadoc. The javadoc tool did not generate any warning
messages.
[exec]
[exec] +1 javac. The applied patch does not increase the total number
of javac compiler warnings.
[exec]
[exec] +1 findbugs. The patch does not introduce any new Findbugs
warnings.
[exec]
[exec] +1 release audit. The applied patch does not increase the
total number of release audit warnings.
Unit test:
all pass
Thanks,
Daniel
> Wrong alias assinged to store operator
> --------------------------------------
>
> Key: PIG-1962
> URL: https://issues.apache.org/jira/browse/PIG-1962
> Project: Pig
> Issue Type: Bug
> Components: impl
> Affects Versions: 0.9.0
> Reporter: Richard Ding
> Assignee: Daniel Dai
> Fix For: 0.9.0
>
> Attachments: PIG-1962-1.patch, PIG-1962-2.patch
>
>
> Given the script
> {code}
> A = load 'data' as (a0, a1);
> B = filter A by a0 > 1;
> store B into 'output';
> {code}
> Alias 'A' (instead of 'B') is assigned to the store operator:
> {code}
> Map Plan
> A: Store(file:///Users/trunk/output:org.apache.pig.builtin.PigStorage) -
> scope-11
> |
> |---A: New For Each(false,false)[bag] - scope-10
> | |
> | Project[bytearray][0] - scope-6
> | |
> | Project[bytearray][1] - scope-8
> |
> |---B: Filter[bag] - scope-1
> | |
> | Greater Than[boolean] - scope-5
> | |
> | |---Cast[int] - scope-3
> | | |
> | | |---Project[bytearray][0] - scope-2
> | |
> | |---Constant(1) - scope-4
> |
> |---A:
> Load(file:///Users/trunk/data:org.apache.pig.builtin.PigStorage) -
> scope-0--------
> {code}
>
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
