You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by goiriX <el...@gmail.com> on 2014/03/07 20:22:14 UTC
LimitOptimizer
Hi,
I was checking the code for the LimitOptimizer and there's a piece of code
that is not very clear. I'm not sure if it's a bug or just an indentation
issue. There's a closing bracket at the end without indentation and this
comes from a check on the limitPlan
(src/org/apache/pig/newplan/logical/rules/LimitOptimizer.java line 142),
which doesn't match the style of the rest of the code. Is the code right as
is or is there an error?
I wanted to double check before reporting a bug.
Thanks,
Inigo
Re: LimitOptimizer
Posted by Daniel Dai <da...@hortonworks.com>.
This is introduced by PIG-1926 for limit by scalar. It also looks like
a bug to me. Feel free to file a ticket and investigate it.
Thanks,
Daniel
On Fri, Mar 7, 2014 at 11:22 AM, goiriX <el...@gmail.com> wrote:
> Hi,
> I was checking the code for the LimitOptimizer and there's a piece of code
> that is not very clear. I'm not sure if it's a bug or just an indentation
> issue. There's a closing bracket at the end without indentation and this
> comes from a check on the limitPlan
> (src/org/apache/pig/newplan/logical/rules/LimitOptimizer.java line 142),
> which doesn't match the style of the rest of the code. Is the code right as
> is or is there an error?
>
> I wanted to double check before reporting a bug.
>
> Thanks,
> Inigo
--
CONFIDENTIALITY NOTICE
NOTICE: This message is intended for the use of the individual or entity to
which it is addressed and may contain information that is confidential,
privileged and exempt from disclosure under applicable law. If the reader
of this message is not the intended recipient, you are hereby notified that
any printing, copying, dissemination, distribution, disclosure or
forwarding of this communication is strictly prohibited. If you have
received this communication in error, please contact the sender immediately
and delete it from your system. Thank You.