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 <billgra...@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 billgra...@gmail.com going forward.*