[
https://issues.apache.org/jira/browse/PIG-1195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12802552#action_12802552
]
Pradeep Kamath commented on PIG-1195:
-------------------------------------
One comment:
- In unit test I would recommend you use Util.createInputFile() method which
take the minicluster as an input arg and creates a input file on the cluster.
Also delete the file in a finally using Util.deleteFile() - the
Util.generateURI() is something which will create problems while merging to
load-store-redesign branch.
Otherwise +1
> POSort should take care of sort order
> -------------------------------------
>
> Key: PIG-1195
> URL: https://issues.apache.org/jira/browse/PIG-1195
> Project: Pig
> Issue Type: Bug
> Components: impl
> Affects Versions: 0.6.0
> Reporter: Daniel Dai
> Assignee: Daniel Dai
> Fix For: 0.6.0
>
> Attachments: PIG-1195-1.patch, PIG-1195-2.patch, PIG-1195-3.patch
>
>
> POSort always use ascending order. We shall obey the sort order as specified
> in the script.
> For example, the following script does not do the right thing if we turn off
> secondary sort (which means, we will rely on POSort to sort):
> {code}
> A = load 'input' as (a0:int);
> B = group A ALL;
> C = foreach B {
> D = order A by a0 desc;
> generate D;
> };
> dump C;
> {code}
> If we run it using the command line "java -Xmx512m
> -Dpig.exec.nosecondarykey=true -jar pig.jar 1.pig".
> The sort order for D is ascending.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
