> On Jan. 22, 2014, 9:43 p.m., Daniel Dai wrote: > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigHadoopLogger.java, > > line 68 > > <https://reviews.apache.org/r/17087/diff/1/?file=429825#file429825line68> > > > > New LoadFunc/StoreFunc might use it, right?
I mean, LoadFunc/StoreFunc does not currently use PigHadoopLogger. We can do another jira to fix that. - Aniket ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17087/#review32558 ----------------------------------------------------------- On Jan. 22, 2014, 7:24 p.m., Aniket Mokashi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17087/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2014, 7:24 p.m.) > > > Review request for pig, Cheolsoo Park, Daniel Dai, and Thejas Nair. > > > Bugs: PIG-2207 > https://issues.apache.org/jira/browse/PIG-2207 > > > Repository: pig-git > > > Description > ------- > > - Grouped counters for udfs > - Log at least once. > > > Diffs > ----- > > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigHadoopLogger.java > 6842b10 > > Diff: https://reviews.apache.org/r/17087/diff/ > > > Testing > ------- > > > Thanks, > > Aniket Mokashi > >