<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<style type="text/css" style="display:none"><!--P{margin-top:0;margin-bottom:0;} .ms-cui-menu {background-color:#ffffff;border:1px rgb(166, 166, 166) solid;font-family:"Segoe UI","Segoe WP","Segoe UI WPC",Tahoma,Arial,sans-serif;font-size:10pt;color:rgb(51, 51, 51);} .ms-cui-menusection-title {display:none;} .ms-cui-ctl {vertical-align:text-top;text-decoration:none;color:rgb(51, 51, 51);} .ms-cui-ctl-on {background-color:rgb(205, 230, 247);opacity: 0.8;} .ms-cui-img-cont-float {display:inline-block;margin-top:2px} .ms-cui-smenu-inner {padding-top:0px;} .ms-owa-paste-option-icon {margin: 0px 6px 0px 6px;vertical-align:middle!important;padding-bottom: 2px;display:inline-block;} .ms-rtePasteFlyout-option:hover {background-color:rgb(205, 230, 247) !important;opacity:1 !important;} .ms-rtePasteFlyout-option {padding:8px 4px 8px 4px;outline:none;} .ms-cui-menusection {float:left; width:85px;height:24px;overflow:hidden}.wf {speak:none; font-weight:normal; font-variant:normal; text-transform:none; -webkit-font-smoothing:antialiased; vertical-align:middle; display:inline-block;}.wf-family-owa {font-family:'o365Icons'}@font-face { font-family:'o365IconsIE8'; src:url('prem/15.0.995.29/resources/styles/office365icons.ie8.eot?#iefix') format('embedded-opentype'), url('prem/15.0.995.29/resources/styles/office365icons.ie8.woff') format('woff'), url('prem/15.0.995.29/resources/styles/office365icons.ie8.ttf') format('truetype'); font-weight:normal; font-style:normal;}@font-face { font-family:'o365IconsMouse'; src:url('prem/15.0.995.29/resources/styles/office365icons.mouse.eot?#iefix') format('embedded-opentype'), url('prem/15.0.995.29/resources/styles/office365icons.mouse.woff') format('woff'), url('prem/15.0.995.29/resources/styles/office365icons.mouse.ttf') format('truetype'); font-weight:normal; font-style:normal;}.wf-family-owa {font-family:'o365IconsMouse'}.ie8 .wf-family-owa {font-family:'o365IconsIE8'}.ie8 .wf-owa-play-large:before {content:'\e254';}.notIE8 .wf-owa-play-large:before {content:'\e054';}.ie8 .wf-owa-play-large {color:#FFFFFF/*$WFWhiteColor*/;}.notIE8 .wf-owa-play-large {border-color:#FFFFFF/*$WFWhiteColor*/; width:1.4em; height:1.4em; border-width:.1em; border-style:solid; border-radius:.8em; text-align:center; box-sizing:border-box; -moz-box-sizing:border-box; padding:0.1em; color:#FFFFFF/*$WFWhiteColor*/;}.ie8 .wf-size-play-large {width:40px; height:40px; font-size:30px}.notIE8 .wf-size-play-large {width:40px; height:40px; font-size:30px}.notIE8 .wf-owa-triangle-down-small:before {content:'\e052';}.ie8 .wf-owa-triangle-down-small:before { content:'\e052';}.ie8 .wf-owa-triangle-down-small {color:#666666/*$WFGreyColor*/;}.wf-size-x20 {font-size: 20px!important;}--></style>
</head>
<body dir="ltr" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;">
<p>Hi Antoine,<br>
</p>
<p><br>
</p>
<p>We're using mutable event payloads to support extensibility of Camel routes AOP in our Camel CDI extension.</p>
<p><br>
</p>
<p>Antonin<br>
</p>
<div style="color: rgb(33, 33, 33);">
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" color="#000000" face="Calibri, sans-serif"><b>From:</b> cdi-dev-bounces@lists.jboss.org <cdi-dev-bounces@lists.jboss.org> on behalf of John D. Ament <johndament@apache.org><br>
<b>Sent:</b> Tuesday, December 16, 2014 7:30 PM<br>
<b>To:</b> Thorben Janssen; cdi-dev<br>
<b>Subject:</b> Re: [cdi-dev] Is the concept of mutable event payload specified</font>
<div> </div>
</div>
<div>Agreed.. I have mutable events all over my current code base.<br>
<br>
<div class="gmail_quote">On Tue Dec 16 2014 at 1:22:05 PM Thorben Janssen <<a href="mailto:thjanssen123@gmail.com">thjanssen123@gmail.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">
<div dir="ltr">We should be careful with making the payload imutable. I know of several applications (and companies) that change the payload to send some information back to to the event producer.
<div>We shouldn't break these apps, if we can solve it in a different way.</div>
</div>
<div class="gmail_extra"><br clear="all">
<div>
<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>
</div>
</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">2014-12-16 18:46 GMT+01:00 Romain Manni-Bucau <span dir="ltr">
<<a href="mailto:rmannibucau@gmail.com" target="_blank">rmannibucau@gmail.com</a>></span>:
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">
Hi Antoine,<br>
<br>
why isn't it portable?<br>
<span><font color="#888888"><br>
<br>
Romain Manni-Bucau<br>
@rmannibucau<br>
<a href="http://www.tomitribe.com" target="_blank">http://www.tomitribe.com</a><br>
<a href="http://rmannibucau.wordpress.com" target="_blank">http://rmannibucau.wordpress.com</a><br>
<a href="https://github.com/rmannibucau" target="_blank">https://github.com/rmannibucau</a><br>
</font></span>
<div>
<div><br>
<br>
2014-12-16 18:39 GMT+01:00 Pete Muir <<a href="mailto:pmuir@redhat.com" target="_blank">pmuir@redhat.com</a>>:<br>
> I don’t think it’s specified. As objects are, by default in Java, mutable, I<br>
> would assume that payloads are implicitly mutable.<br>
><br>
> On 16 Dec 2014, at 16:31, Antoine Sabot-Durand <<a href="mailto:antoine@sabot-durand.net" target="_blank">antoine@sabot-durand.net</a>><br>
> wrote:<br>
><br>
> Hi guys,<br>
><br>
><br>
> Always working on Async event concept and discussion around mutable<br>
> payloads. I was looking where in the spec we specified the fact that fired<br>
> payload are mutable. I red-read chapter 10<br>
> (<a href="http://docs.jboss.org/cdi/spec/1.2/cdi-spec.html#events" target="_blank">http://docs.jboss.org/cdi/spec/1.2/cdi-spec.html#events</a>) twice and didn’t<br>
> found. I also browsed JIRA and TCK to find any ref to this feature and found<br>
> nothing. On the other hand it is not specified that payload should be<br>
> immutable ;)<br>
><br>
> I’d be happy if some of you could have a look and see if I missed something.<br>
><br>
> If I’m not wrong, the mutable payload we (including myself) advertise in CDI<br>
> is a non portable feature (I’m the firs surprised here). So I propose that :<br>
><br>
> 1) We decide to write something in the specification about allowing or<br>
> forbidding it (I know some people not happy with this mix between observer<br>
> and visitor pattern)<br>
> 1bis) Should we decide to forbid it by default, we should provide an<br>
> alternative mode to allow people using this unspecified feature<br>
> 2) Forbid it for fireAsync()<br>
><br>
><br>
> Thanks for your feedback and your correction if I missed the feature in the<br>
> spec.<br>
><br>
> Antoine<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<br>
> under the Apache License, Version 2<br>
> (<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<br>
> provided on this list, the provider waives all patent and other intellectual<br>
> property rights inherent in such information.<br>
><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<br>
> under the Apache License, Version 2<br>
> (<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<br>
> provided on this list, the provider waives all patent and other intellectual<br>
> property rights inherent in such information.<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.</div>
</div>
</blockquote>
</div>
</div>
______________________________<u></u>_________________<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/<u></u>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/<u></u>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>
</div>
</div>
</body>
</html>