commit 25a2d94974c7befd13f90e52b61e297c31ae52e9

[simo]

On Wed, 2010-09-15 at 22:39 +0200, Björn Jacke wrote:
> On 2010-09-15 at 16:17 -0400 simo sent off:
> > lp_printcap_cache_time() is used at a higher level for the automatic
> > reload, I am not sure it is correct to check it here too. This function
> > is meant to avoid rapid firing refreshes but not to delay them for very
> > long times.
> 
> es, it was used in smbd/process.c - but just because there was no other level
> of caching available. The caching of rintcap per client was broken by design,
> see bug 6448 for that. This is exactly the place where lp_printcap_cache_time
> should be placed, it should be obsolete in smbd/process.c then. Without using
> printcap cahe time at this place a large print server will pull all printers
> every second which isn't fun if you have a cups server with hundreds of
> printer. ;-)

Yes, you may have a point here :)

> > Unless you have checked that there are no side effects in changing the
> > semantics of this function I would just apply the time_mono() part of
> > the fix.
> 
> okay, I'll split it it up and just check in the monotonic time part now and do
> some more testing with the printcap caching and the changes that should also be
> made in smbd/process.c later on.

Thanks, I am not opposed in principle to lp_printcap_cache_time() being
used there, but I like to have it dealt with as a separate issue.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Principal Software Engineer at Red Hat, Inc. <simo at redhat.com>