[cdi-dev] [VOTE] Using @Priority to order events instead of adding a parameter in @Observes

[Werner Keil]

AFAIK that is not part of the JDK, thus it should make it easier to ask
them for a MR, last happened about a year ago:
https://jcp.org/en/jsr/detail?id=250




On Tue, Oct 28, 2014 at 12:33 PM, Pete Muir <pmuir at redhat.com> wrote:

> I would be +1 if we can get a commitment to update the version of JSR-250
> shipped in the JDK updated as well, otherwise -1
>
> On 28 Oct 2014, at 10:13, Werner Keil <werner.keil at gmail.com> wrote:
>
> +1 for 1)
>
> Unlike @Inject the Maven JAR for JSR-250 is a bit bigger (~20kb) but there
> are existing dependencies that are not part of the JDK, most notably
> JSR-330.
>
> Not sure, if subpackages like "security" or "sql" under 250 matter at all,
> if not, we could explore if the ideas for "stripping" libraries proposed by
> Oracle may also work for SE/EE. This was discussed by OpenJDK architects
> including Mark Reinhold with the EC. So far no real progress on that, but
> till this JSR goes final or EE 8 it could work to get dependencies a bit
> lighter, too.
>
> It is likely, some annotation JSRs not just 250 need overhaul, e.g. to
> finally make use of JSR-308, so an MR for 250 could be cumbersome, but
> seems much easier here than e.g. bringing JSR-305 back to life;-)
>
> Werner
>
>
> On Tue, Oct 28, 2014 at 9:58 AM, <cdi-dev-request at lists.jboss.org> wrote:
>
>> Send cdi-dev mailing list submissions to
>>         cdi-dev at lists.jboss.org
>>
>> To subscribe or unsubscribe via the World Wide Web, visit
>>         https://lists.jboss.org/mailman/listinfo/cdi-dev
>> or, via email, send a message with subject or body 'help' to
>>         cdi-dev-request at lists.jboss.org
>>
>> You can reach the person managing the list at
>>         cdi-dev-owner at lists.jboss.org
>>
>> When replying, please edit your Subject line so it is more specific
>> than "Re: Contents of cdi-dev digest..."
>>
>>
>> Today's Topics:
>>
>>    1. Re: microbenchmark for CDI performance (Mohan Radhakrishnan)
>>    2. [VOTE] Using @Priority to order events instead of adding  a
>>       parameter in @Observes (Antoine Sabot-Durand)
>>    3. No meeting tomorrow (Antoine Sabot-Durand)
>>    4. Updated Invitation: CDI weekly meeting @ Weekly from 18:00 to
>>       19:00 on Wednesday except Wed 1 Oct 18:00, Wed 15 Oct 18:00 or
>>       Wed 29 Oct 18:00 (ASD Perso) (antoine at sabot-durand.net)
>>
>>
>> ----------------------------------------------------------------------
>>
>> Message: 2
>> Date: Tue, 28 Oct 2014 09:56:14 +0100
>> From: Antoine Sabot-Durand <antoine at sabot-durand.net>
>> Subject: [cdi-dev] [VOTE] Using @Priority to order events instead of
>>         adding  a parameter in @Observes
>> To: cdi-dev <cdi-dev at lists.jboss.org>
>> Message-ID: <ECAAD9B3-649C-4856-BB24-82AB9EF41763 at sabot-durand.net>
>> Content-Type: text/plain; charset=utf-8
>>
>> To add events ordering feature we have two choices:
>>
>> 1) Use @Priority from common annotation specification (JSR-250)  (vote +1)
>> pros:
>> - more consistent with other ordering mechanism we already have in CDI
>> (Interceptors, decorators, Alternatives)
>> - more Java EE consistent
>>
>> cons:
>> - time consumed in JSR 250 MR participation (@Priority must be update to
>> support parameter for target)
>> - as @Priority is not part of Java SE, CDI light on java SE will have to
>> add the jsr250 jar as dependency only to have this annotation (which will
>> make it a little less light)
>>
>> 2) Add a parameter to @Observes annotation to give order to an observer
>> (vote -1)
>> pros:
>> - works out of the box (we can add this property without ask for other
>> specs modification)
>> - avoid two annotations to declare an ordered observer (simpler usage)
>>
>> cons:
>> - less Java EE spirit
>> - could be seen as an inconsistency with the way we order Interceptors,
>> Decorators and Alternatives.
>>
>> ????????????????????
>>
>> Who can vote?  Everybody registered to this ML can vote. All votes will
>> be binding
>> How to vote? Answer this mail and vote +1 (for @Priority) 0 or -1(for
>> parameter in @Observes)
>> Vote will be closed in 72 hours
>>
>
> _______________________________________________
> cdi-dev mailing list
> cdi-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/cdi-dev
>
> Note that for all code provided on this list, the provider licenses the
> code under the Apache License, Version 2 (
> http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas
> provided on this list, the provider waives all patent and other
> intellectual property rights inherent in such information.
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/cdi-dev/attachments/20141028/ce084f56/attachment.html