You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by Satish Saley <sa...@gmail.com> on 2017/10/03 17:23:50 UTC
Re: Review Request 62422: PIG-4120 Broadcast the index file in case of
POMergeCoGroup and POMergeJoin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62422/
-----------------------------------------------------------
(Updated Oct. 3, 2017, 10:23 a.m.)
Review request for pig.
Bugs: PIG-4120
https://issues.apache.org/jira/browse/PIG-4120
Repository: pig-git
Description
-------
PIG-4120 Broadcast the index file in case of POMergeCoGroup and POMergeJoin
Diffs (updated)
-----
src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMergeCogroup.java f18d47a34
src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMergeCogroupTez.java PRE-CREATION
src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMergeJoin.java 815a32586
src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMergeJoinTez.java PRE-CREATION
src/org/apache/pig/backend/hadoop/executionengine/tez/plan/TezCompiler.java 79739e98a
src/org/apache/pig/backend/hadoop/executionengine/tez/plan/TezOperator.java 6e99d395d
src/org/apache/pig/impl/builtin/DefaultIndexableLoader.java a4688e499
src/org/apache/pig/impl/builtin/TezIndexableLoader.java PRE-CREATION
src/org/apache/pig/tools/pigstats/tez/TezScriptState.java 716f079b6
test/org/apache/pig/test/data/GoldenFiles/tez/TEZC-MergeCogroup-1.gld PRE-CREATION
test/org/apache/pig/test/data/GoldenFiles/tez/TEZC-MergeJoin-1.gld PRE-CREATION
test/org/apache/pig/tez/TestTezCompiler.java f99d6f39c
Diff: https://reviews.apache.org/r/62422/diff/5/
Changes: https://reviews.apache.org/r/62422/diff/4-5/
Testing
-------
Thanks,
Satish Saley
Re: Review Request 62422: PIG-4120 Broadcast the index file in case of
POMergeCoGroup and POMergeJoin
Posted by Rohini Palaniswamy <ro...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62422/#review186981
-----------------------------------------------------------
Ship it!
Ship It!
- Rohini Palaniswamy
On Oct. 3, 2017, 5:23 p.m., Satish Saley wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62422/
> -----------------------------------------------------------
>
> (Updated Oct. 3, 2017, 5:23 p.m.)
>
>
> Review request for pig.
>
>
> Bugs: PIG-4120
> https://issues.apache.org/jira/browse/PIG-4120
>
>
> Repository: pig-git
>
>
> Description
> -------
>
> PIG-4120 Broadcast the index file in case of POMergeCoGroup and POMergeJoin
>
>
> Diffs
> -----
>
> src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMergeCogroup.java f18d47a34
> src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMergeCogroupTez.java PRE-CREATION
> src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMergeJoin.java 815a32586
> src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMergeJoinTez.java PRE-CREATION
> src/org/apache/pig/backend/hadoop/executionengine/tez/plan/TezCompiler.java 79739e98a
> src/org/apache/pig/backend/hadoop/executionengine/tez/plan/TezOperator.java 6e99d395d
> src/org/apache/pig/impl/builtin/DefaultIndexableLoader.java a4688e499
> src/org/apache/pig/impl/builtin/TezIndexableLoader.java PRE-CREATION
> src/org/apache/pig/tools/pigstats/tez/TezScriptState.java 716f079b6
> test/org/apache/pig/test/data/GoldenFiles/tez/TEZC-MergeCogroup-1.gld PRE-CREATION
> test/org/apache/pig/test/data/GoldenFiles/tez/TEZC-MergeJoin-1.gld PRE-CREATION
> test/org/apache/pig/tez/TestTezCompiler.java f99d6f39c
>
>
> Diff: https://reviews.apache.org/r/62422/diff/5/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Satish Saley
>
>