[illumos-Developer] Code review: Feature #677 - Allow adjustment of cache timeout in idmapd

Garrett D'Amore garrett at nexenta.com
Sat Jan 29 19:35:49 PST 2011


Looks pretty good to me.  I wonder where the 10 minutes and 1 hour came
from though.  They feel like they should be placed as macros in a common
header with other compile-time defaults (if any).

	- Garrett

On Sat, 2011-01-29 at 16:26 -0500, Gordon Ross wrote:
> Please review the implementation of:
> 
> Feature #677: Allow adjustment of cache timeout in idmapd
> https://www.illumos.org/issues/677
> 
> Here's the webrev:
> http://cr.illumos.org/view/xjhhzxeh/
> 
> Thanks,
> Gordon
> 
> _______________________________________________
> Developer mailing list
> Developer at lists.illumos.org
> http://lists.illumos.org/m/listinfo/developer





More information about the Developer mailing list