Hi, On Tue, Oct 13, 2015 at 05:45:58PM +0300, Lev Stipakov wrote: > When server exits / restarts (gets SIGUSR1, SIGTERM, SIGHUP, SIGINT) and > explicit-exit-notify is set, server sends RESTART control channel command to > all clients and reschedules received signal in 2 secs. > > When client receives RESTART command, it either reconnects to the same server > or > advances to the new one, depends on parameter comes with RESTART > command - behavior is controlled by explicit-exit-notify in the server config. > > v3: > - Use control channel "RESTART" command instead of new OCC code to > notify clients > - Configure on the server side (by value of explicit-exit-notify) if > client should reconnect to the same server or advance to the next one > - Fix compilation when OCC is disabled (--enable-small) > - Update man page
Thanks for this. A few observations...
- it does not apply anymore, due to new code in multi.h
(someone committed #ifdef ENABLE_ASYNC_PUSH in the meantime :-) ),
so I'd appreciate a rebase
- I'm not totally happy about the #ifdef ENABLE_OCC and the fact that
multi_push_restart_schedule_exit() is not #ifdef'ed as well (a smart
compiler will optimize it away, as it is static and not called, but
I'm not sure whether it is considered good style).
Since the new server side code does not actually *do* OCC any more we
are just #ifdef'ing it to access options->ce.explicit_exit_notify
because that one is only compiled in #ifdef ENABLE_OCC ... so we're
coupling this new functionality to an #ifdef which is not really
relevant.
No good suggestion yet, just expressing thoughts...
gert
--
USENET is *not* the non-clickable part of WWW!
//www.muc.de/~gert/
Gert Doering - Munich, Germany g...@greenie.muc.de
fax: +49-89-35655025 g...@net.informatik.tu-muenchen.de
signature.asc
Description: PGP signature
