<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">+1 (for @Priority) </span></div><div class="gmail_extra"><br clear="all"><div><div dir="ltr"><div>--</div><div><b>Thorben Janssen</b><br></div><div><br></div><div><a href="https://twitter.com/thjanssen123" target="_blank">@thjanssen123</a><br></div><a href="http://www.thoughts-on-java.org" target="_blank">www.thoughts-on-java.org</a> <br></div></div>
<br><div class="gmail_quote">2014-10-28 10:00 GMT+01:00 Antonio Goncalves <span dir="ltr"><<a href="mailto:antonio.goncalves@gmail.com" target="_blank">antonio.goncalves@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span style="font-family:arial,sans-serif;font-size:12.7272720336914px">+1 (for @Priority) </span><br></div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Tue, Oct 28, 2014 at 9: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" target="_blank">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><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div dir="ltr">Antonio Goncalves <br>Software architect, Java Champion and Pluralsight author<br><br><a href="http://www.antoniogoncalves.org" target="_blank">Web site</a> | <a href="http://twitter.com/agoncal" target="_blank">Twitter</a> | <a href="http://www.linkedin.com/in/agoncal" target="_blank">LinkedIn</a> | <a href="http://pluralsight.com/training/Authors/Details/antonio-goncalves" target="_blank">Pluralsight</a> | <a href="http://www.parisjug.org" target="_blank">Paris JUG</a> | <a href="http://www.devoxx.fr" target="_blank">Devoxx France</a></div>
</font></span></div>
<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.<br></blockquote></div><br></div>