Hi All,

SendThread is exitting after OOM(ZooKeeper-1100):-

After reviewing the ClientCnxn code, there is still chances of exiting the SendThread without intimating the users.

Say if client throws OOMError and entered into the throwable block. Here again while sending the Disconnected event, its creating "new WatchedEvent()" object.

This will throw OOMError and leads to exit the SendThread without any Disconnected event notification.

Earlier we had a discussion reg the OOMError in SendThread. But we didn't notice these and closed the issue.

try{

    //...

} catch (Throwable e)

{

    //..

    cleanup();

   if(state.isAlive()){

        eventThread.queueEvent(

        new WatchedEvent(Event.EventType.None,  Event.KeeperState.Disconnected, null) )

   }

   //....

}

Thanks & Regards,

Rakesh R

Can you open a new issue with this?

Thanks,
C

On Tue, Jan 24, 2012 at 4:36 AM, Rakesh R <[EMAIL PROTECTED]> wrote:
> Hi All,
>
>
>
> SendThread is exitting after OOM(ZooKeeper-1100):-
>
>
>
> After reviewing the ClientCnxn code, there is still chances of exiting the SendThread without intimating the users.
>
> Say if client throws OOMError and entered into the throwable block. Here again while sending the Disconnected event, its creating "new WatchedEvent()" object.
>
> This will throw OOMError and leads to exit the SendThread without any Disconnected event notification.
>
>
>
> Earlier we had a discussion reg the OOMError in SendThread. But we didn't notice these and closed the issue.
>
>
>
> try{
>
>    //...
>
> } catch (Throwable e)
>
> {
>
>    //..
>
>    cleanup();
>
>   if(state.isAlive()){
>
>        eventThread.queueEvent(
>
>        new WatchedEvent(Event.EventType.None,  Event.KeeperState.Disconnected, null) )
>
>   }
>
>   //....
>
> }
>
>
>
>
>
> Thanks & Regards,
>
> Rakesh R