[PATCH] Cache: gracefully exit the cache manager process
mdounin at mdounin.ru
Tue Nov 5 00:06:05 UTC 2013
On Mon, Nov 04, 2013 at 02:07:23PM -0800, Yichun Zhang (agentzh) wrote:
> On Mon, Nov 4, 2013 at 1:31 PM, Maxim Dounin wrote:
> > The cache manager process isn't worker process, so calling
> > ngx_worker_process_exit() looks strange.
> But ngx_cache_manager_process_cycle() is already calling
Yes, and this is known to cause problems. E.g., more than one
module was broken due to init_process callback being called in
cache manager/loader processes which are quite different from
> Are you suggesting removing this call as well?
No, but it is something we may consider.
In any case, I don't think that adding calls to worker-related
stuff is a good idea, at least without a thoughtful analysis of
possible side effects.
More information about the nginx-devel