[jira] [Commented] (PIG-3804) Limit Optimizer getLimitPlan broken

["Aniket Mokashi (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/PIG-3804?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14010421#comment-14010421 ] 

Aniket Mokashi commented on PIG-3804:
-------------------------------------

Code looks good to me. It's just indented badly.

> Limit Optimizer getLimitPlan broken
> -----------------------------------
>
>                 Key: PIG-3804
>                 URL: https://issues.apache.org/jira/browse/PIG-3804
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: 0.8.0, 0.8.1, 0.9.2, 0.10.0, 0.11, 0.10.1, 0.12.0, 0.11.1
>            Reporter: Íñigo Goiri
>            Priority: Minor
>             Fix For: 0.14.0
>
>
> The LimitOptimizer cases don't seem right (Starting at line 142 in file src/org/apache/pig/newplan/logical/rules/LimitOptimizer.java). It looks like one optimization was added wrongly and one extra bracket had to be added at the end. There's a TODO note but it looks like some parts weren't commented properly.



--
This message was sent by Atlassian JIRA
(v6.2#6252)