On Fri, Mar 23, 2012 at 2:39 PM, Ben Pfaff <b...@nicira.com> wrote: > That's fine with. Omitting such files would also be fine, I guess. > But the new behavior appeared, at a glance, to be that we don't catch > exceptions in os.stat() at all. No? > > No. I have added that check in the latest patch I sent.
> On Fri, Mar 23, 2012 at 02:37:21PM -0700, Raju Subramanian wrote: > > The old behaviour was if os.stat() failed, it would still add the file to > > the report but just wouldn't update the counter tracking the sizes. Do > you > > want to maintain this behaviour? > > > > On Fri, Mar 23, 2012 at 2:26 PM, Ben Pfaff <b...@nicira.com> wrote: > > > > > On Fri, Mar 23, 2012 at 02:22:23PM -0700, Raju Subramanian wrote: > > > > When size limit is reached in the middle of processing a dir, > > > > the report ends up containing oldest files. This change adds > > > > an optional param in the plugin to prioritize newer files. > > > > > > > > Feature #9937 > > > > Requested-by: Ronald Lee <r...@nicira.com> > > > > Signed-off-by: Raju Subramanian <rsubraman...@nicira.com> > > > > > > The new version still doesn't catch exceptions in os.stat. > > > > > > Thanks, > > > > > > Ben. > > > >
_______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
