You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by Bill Graham <bi...@gmail.com> on 2012/03/26 07:07:27 UTC
LoadMetada.getStatistics() is unsupported
Hi,
We found a bug where InterStorage.getStatistics() was throwing
UnsupportedOperationException which was causing problems for
JobControlCompiler (see
https://issues.apache.org/jira/browse/PIG-2616<https://issues.apache.org/jira/browse/PIG-2616>).
PIG-2616 changes JobControlCompiler to catch the exception, but I'm
wondering why InterStorage implements LoadMetada but getStatistics() is
unsupported? Is there a good reason for this, or should we implement that
method?
thanks,
Bill
Re: LoadMetada.getStatistics() is unsupported
Posted by Bill Graham <bi...@gmail.com>.
Sounds good, I just updated the patch. Return null is what the LoadMetadata
javadoc says to do when stats aren't available actually, so the null case
should be expected.
On Mon, Mar 26, 2012 at 1:03 AM, Daniel Dai <da...@hortonworks.com> wrote:
> Return null should be fine. I changed that for BinStorage, which
> otherwise breaks unit test.
>
> Daniel
>
> On Sun, Mar 25, 2012 at 10:07 PM, Bill Graham <bi...@gmail.com>
> wrote:
> > Hi,
> >
> > We found a bug where InterStorage.getStatistics() was throwing
> > UnsupportedOperationException which was causing problems for
> > JobControlCompiler (see
> > https://issues.apache.org/jira/browse/PIG-2616<
> https://issues.apache.org/jira/browse/PIG-2616>).
> > PIG-2616 changes JobControlCompiler to catch the exception, but I'm
> > wondering why InterStorage implements LoadMetada but getStatistics() is
> > unsupported? Is there a good reason for this, or should we implement that
> > method?
> >
> > thanks,
> > Bill
>
--
*Note that I'm no longer using my Yahoo! email address. Please email me at
billgraham@gmail.com going forward.*
Re: LoadMetada.getStatistics() is unsupported
Posted by Daniel Dai <da...@hortonworks.com>.
Return null should be fine. I changed that for BinStorage, which
otherwise breaks unit test.
Daniel
On Sun, Mar 25, 2012 at 10:07 PM, Bill Graham <bi...@gmail.com> wrote:
> Hi,
>
> We found a bug where InterStorage.getStatistics() was throwing
> UnsupportedOperationException which was causing problems for
> JobControlCompiler (see
> https://issues.apache.org/jira/browse/PIG-2616<https://issues.apache.org/jira/browse/PIG-2616>).
> PIG-2616 changes JobControlCompiler to catch the exception, but I'm
> wondering why InterStorage implements LoadMetada but getStatistics() is
> unsupported? Is there a good reason for this, or should we implement that
> method?
>
> thanks,
> Bill