Deprecating ChannelPipelineCoverage?
Iain McGinniss
iainmcgin at gmail.com
Mon Nov 9 05:45:41 EST 2009
How about @SharedHandler and @UnsharedHandler? @Shared describes
Mike's usage, where he has a handler used by multiple pipelines but
within which he has implemented the appropriate logic to avoid
threading issues.
A side question: does netty actually check these annotations at
runtime, to make sure an unshared handler instance is not used by
multiple pipelines?
Iain
On 9 Nov 2009, at 00:12, Trustin Lee (이희승) wrote:
> Good point. Then, do you have some suggestion for a better name than
> ChannelPipelineCoverage, or do you think it's just fine? I think
> there might be a better name that makes it easier to understand.
>
> Thanks
>
> — Trustin Lee, http://gleamynode.net/
>
>
>
> On Mon, Nov 9, 2009 at 7:00 AM, Mike McGrady
> <mmcgrady at topiatechnology.com> wrote:
>> Hi, Trustin,
>>
>> I think that the present annotations are more explicit. The present
>> annotations say what is actually happening. The stateful and
>> stateless annotations would not actually fit what is happening with
>> my
>> code. I use stateful code with "all" and this is not a problem. If
>> the annotation were changed I would have "stateless" as the
>> annotation
>> but have a stateful handler.
>>
>> Mike
>>
>> On Nov 8, 2009, at 1:24 PM, Trustin Lee (이희승) wrote:
>>
>>> What do you think about replacing ChannelPipelineCoverage with more
>>> explicit annotations such as @Stateful and @Stateless?
>>>
>>> ChannelPipelineCoverage has a confusing name (at least for a
>>> beginner), and its value cannot be validated on compile time because
>>> it's string.
>>>
>>> — Trustin Lee, http://gleamynode.net/
>>>
>>> _______________________________________________
>>> netty-dev mailing list
>>> netty-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/netty-dev
>>
>> Mike McGrady
>> Principal Investigator AF081-028 AFRL SBIR
>> Senior Engineer
>> Topia Technology, Inc.
>> 1.253.720.3365
>> mmcgrady at topiatechnology.com
>>
>>
>>
>>
>>
>>
>>
>>
>>
>>
>> _______________________________________________
>> netty-dev mailing list
>> netty-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/netty-dev
>>
>
> _______________________________________________
> netty-dev mailing list
> netty-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/netty-dev
More information about the netty-dev
mailing list