<div dir="ltr">-1 (vote for a param in @Observes)<div><br></div><div>The reason being, it sounds like there's a lot of unknowns/gaps with trying to do #1. Plus only part of 250 ships with the JVM, as the rest of it comes from the EE spec: <a href="https://javaee-spec.java.net/nonav/javadocs/javax/annotation/Priority.html">https://javaee-spec.java.net/nonav/javadocs/javax/annotation/Priority.html</a></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 28, 2014 at 4:56 AM, Antoine Sabot-Durand <span dir="ltr"><<a href="mailto:antoine@sabot-durand.net" target="_blank">antoine@sabot-durand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">To add events ordering feature we have two choices:<br>
<br>
1) Use @Priority from common annotation specification (JSR-250) (vote +1)<br>
pros:<br>
- more consistent with other ordering mechanism we already have in CDI (Interceptors, decorators, Alternatives)<br>
- more Java EE consistent<br>
<br>
cons:<br>
- time consumed in JSR 250 MR participation (@Priority must be update to support parameter for target)<br>
- 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)<br>
<br>
2) Add a parameter to @Observes annotation to give order to an observer (vote -1)<br>
pros:<br>
- works out of the box (we can add this property without ask for other specs modification)<br>
- avoid two annotations to declare an ordered observer (simpler usage)<br>
<br>
cons:<br>
- less Java EE spirit<br>
- could be seen as an inconsistency with the way we order Interceptors, Decorators and Alternatives.<br>
<br>
————————————————————<br>
<br>
Who can vote? Everybody registered to this ML can vote. All votes will be binding<br>
How to vote? Answer this mail and vote +1 (for @Priority) 0 or -1(for parameter in @Observes)<br>
Vote will be closed in 72 hours<br>
<br>
<br>
_______________________________________________<br>
cdi-dev mailing list<br>
<a href="mailto:cdi-dev@lists.jboss.org">cdi-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/cdi-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/cdi-dev</a><br>
<br>
Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (<a href="http://www.apache.org/licenses/LICENSE-2.0.html" target="_blank">http://www.apache.org/licenses/LICENSE-2.0.html</a>). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.</blockquote></div><br></div>