Re: Re[2]: [Massiv-crap] Re: [Massiv-devel] Dm bug

[Petr T. <pto...@ss...>]

From: "Ondrej Pecta"


>
> MV>    >Eee, chapu to spravne, ze pri SystemMessage::send() network zavola
> MV>    >callback do node manageru? Jestli jo, tak je zas neco divne v
networku,
> MV>    >protoze jsem se kdysi pri designu callbacku dohodli (a je to i v
dokumentaci
> MV>    >v node manageru), ze se ty callbacky smi volat jen z
Network::next_tick()
> MV>    >prave kvuli takovymdle vecem.
>
> Nic proti chlapi, ale ty 4 callbacky, ktere vola network, tak se
> vzdycky volaji z next_tick()
> jedna se o connection lost / closed + created
> dale o async connection failed.


Problem byl v sendovani message v node mgr, melo by byt fixnuto jak je
zmineno
v jinem mailu.


> Jediny problem, jak Markoid zminuje, je s tim disconnect, ktery je ted
> blbe ( staci si v Global::system_shutdown() zakomentovat to
> close_all_connections( true ). Ale to porad podle mne neni to, o cem
> je tenhle thread.
> Vyresi to vase problemy, ale zase se musi jinak vyresit odpojeni
> klienta z logovaci obrazovky ( kdyz se tam da quit, tak service, ke
> kteremu je pripojen, neni notifikovan o jeho odpojeni a konexi na
> anonymous klienta zavre s connection lost ). Resenim je pri stisku
> QUIT v logovaci obrazovce zavolat close_all_connection( true );


Jukni na
node_manager_client.cpp:NodeManager::Client::disconnect(),
a pripadne ji uprav (asi jen nahradit tim close_all_connection()?).


Petr