Closed: (NETTY-85) Global Channel registry
Trustin Lee
tlee at redhat.com
Tue Dec 2 21:57:23 EST 2008
I've just checked in the DefaultChannelGroup modification. It became
somewhat more complicated, but a user shouldn't notice any difference.
The only difference is that close / unbind / disconnect / setReadable
/ setInterestOps operations are performed in a non-blocking manner for
server socket channels as discussed.
Cheers,
Trustin
On Wed, Dec 03, 2008 at 10:48:06AM +0900, Trustin Lee wrote:
> Hi Frederic,
>
> On Tue, Dec 02, 2008 at 03:36:35PM +0900, Frederic Bregier wrote:
>> Hi Trustin,
>>
>> Hmm, I didn't think about the asynchronous aspect of the close operation.
>> Thanks for pointing this!
>
> You're welcome. Thanks for the feed back!
>
>> Well, I think you're right with proposing a blocking close request.
>> So I think that users should use at least two groups, one for the
>> parent, and one for the children, and then waiting for the first to
>> be really closed before asking the second to close. That's one option.
>> These options should be explicitely written in the doc. WDYT ?
>
> Yes, that's your initial suggestion IIUC.
>
>> Another possibility would be to have the group having two kind of
>> list of channels int it, using the getParent() method.
>> If the getParent() returns null, it is a parent, if not, it is a child.
>> Then when one asked the group to close, it first closes the parent
>> with blocking close, then closes the others channel as usual.
>
> This one is what I suggested in the previous message.
>
> <snip/>
>
>> So that, even if a user makes two groups (one for parent,
>> one for childs), it will block when calling close on the group
>> of parent (awaitUninterruptibly returns immediately).
>> WDYT ?
>
> Sounds great to me. Let me modify the current implementation to perform
> as you described.
>
> Cheers,
>
--
Trustin Lee, Principal Software Engineer, JBoss, a division of Red Hat
--
what we call human nature in actuality is human habit
--
http://gleamynode.net/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
Url : http://lists.jboss.org/pipermail/netty-dev/attachments/20081203/4aebe64b/attachment.bin
More information about the netty-dev
mailing list