> On 2011-07-07 01:16:22, Daniel Dai wrote: > > trunk/src/org/apache/pig/impl/io/FileLocalizer.java, line 796 > > <https://reviews.apache.org/r/976/diff/1/?file=21967#file21967line796> > > > > Is it possible to avoid a listStatus call if we are not use globbing? > > Raghu Angadi wrote: > globStatus() does the check. The check itself is pretty involved. If a > path does not have a glob, it eventually results in only one call to > getFileStatus(). > > Daniel Dai wrote: > Does globStatus check it on local side?
what do mean by 'local side'? globStatus() is client side wrapper over listStatus and getFileStatus. - Raghu ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/976/#review983 ----------------------------------------------------------- On 2011-07-04 04:45:47, Dmitriy Ryaboy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/976/ > ----------------------------------------------------------- > > (Updated 2011-07-04 04:45:47) > > > Review request for pig and Daniel Dai. > > > Summary > ------- > > Posting on behalf of Raghu. > Daniel, can you take a look since this collides directly with your changes in > PIG-1566? > > Argh, reviewboard apparently does not get git diffs. We'll try to make > something it gets and add later.. please see the patch on the Jira in the > meantime. > > > This addresses bug PIG-2142. > https://issues.apache.org/jira/browse/PIG-2142 > > > Diffs > ----- > > trunk/src/org/apache/pig/PigServer.java 1142512 > trunk/src/org/apache/pig/impl/io/FileLocalizer.java 1142512 > trunk/test/org/apache/pig/test/TestPigServer.java 1142512 > > Diff: https://reviews.apache.org/r/976/diff > > > Testing > ------- > > > Thanks, > > Dmitriy > >
