<div dir="ltr">Hi Martin,<div><br></div><div><div><span style="line-height:1.5">yes this particular issue is about concurrent access control. You are right in pointing out that the </span><span style="line-height:1.5">lock should be applied </span></div><div><span style="line-height:1.5">to the whole been and only </span>override-able<span style="line-height:1.5"> on a per method basis (similar to EJB Singleton style locking). </span></div></div><div><br></div><div>Regarding conversation context, its fair enough to point-out that weld allows for configure the conversation lock timeout.</div><div>However this is only true for Weld, this should really be made part of the specification.</div><div><br></div><div>Even if we were to specify a standard way to configure conversation locked timeouts in the CDI specification, it would</div><div>still make the conversation scope the odd one out of the lot. Hence it would be more sensible to design a </div><div>common way to handle concurrent access.</div><div><br></div><div>Also I would argue that you cannot implement a common concurrent access control via interceptors, </div><div>s<span style="line-height:1.5">ince the container will preempt any interceptor based attempt for conversation scoped beans.</span></div><div><span style="line-height:1.5"><br></span></div><div>As Reza pointed out Oracle has no intend to reopen "<span style="line-height:1.5">Concurrency Utilities for Java EE" at this time and is not</span></div><div><span style="line-height:1.5">willing to hand it over to anyone else. The same seems to be true for JTA.</span></div><div><span style="line-height:1.5"><br></span></div><div>Stephan</div><div><span style="line-height:1.5"><br></span></div><div><span style="line-height:1.5"><br></span></div><div><span style="line-height:1.5"><br></span></div><div><span style="line-height:1.5"> </span></div><div><br></div><div><span style="line-height:1.5"><br></span></div><div><span style="line-height:1.5"><br></span></div><div><span style="line-height:1.5"><br></span></div><div><span style="line-height:1.5"><br></span></div><div><br></div><div> </div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, 25 Feb 2016 at 15:50 Reza Rahman <<a href="mailto:reza_rahman@lycos.com">reza_rahman@lycos.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Oracle has pretty much clearly stated it has absolutely no intention of updating the Java EE Concurrency Utilities specification any time soon. My guess is that it will also never allow anyone else to update it either since it owns that specification. If this is a valuable feature to the community (which I definitely think it is) I strongly suggest taking advantage of the fact that this is a gray area and include it in a modular CDI specification so this feature doesn't continue to remain locked into EJB for Java EE users that need to more effectively use things like @Stereotype for service composition.<br>
<br>
> On Feb 25, 2016, at 9:13 AM, Martin Kouba <<a href="mailto:mkouba@redhat.com" target="_blank">mkouba@redhat.com</a>> wrote:<br>
><br>
> Hi Stephan,<br>
><br>
> I like the idea of CDI interceptor solution you're proposing in your<br>
> blogpost [1]. However, concurrency is a difficult topic. First of all,<br>
> this only solves concurrent access to the bean instance (i.e.<br>
> method-level locking) - the bean state is always up to the user. Also<br>
> I'm not so sure it's a good idea to only apply @Lock at the method level<br>
> (some methods are guarded some not - AFAIK EJB does not allow this either).<br>
><br>
> I agree that conversation concurrentAccessTimeout in Weld should be<br>
> configurable. In fact, it should be possible to change this timeout even<br>
> now using Weld API and org.jboss.weld.context.ConversationContext. But<br>
> it should be definitely more straightforward [2].<br>
><br>
> To sum it up - I wouldn't add concurrency control to the spec provided<br>
> it's implementable using interceptors. This is a similar situation as to<br>
> javax.transaction.Transactional and JTA. The best place to specify this<br>
> is IMHO "Concurrency Utilities for Java EE".<br>
><br>
> Martin<br>
><br>
> [1]<br>
> <a href="http://www.knitelius.com/2016/01/25/concurrency-control-for-cdi/" rel="noreferrer" target="_blank">http://www.knitelius.com/2016/01/25/concurrency-control-for-cdi/</a><br>
><br>
> [2]<br>
> <a href="https://issues.jboss.org/browse/WELD-2113" rel="noreferrer" target="_blank">https://issues.jboss.org/browse/WELD-2113</a><br>
><br>
> Dne 24.2.2016 v 20:47 Stephan Knitelius napsal(a):<br>
>> I just want to bring this to everyone attention one more time.<br>
>><br>
>> The conversation scope concurrency control mechanism seems to be a<br>
>> frequent point of pain in many projects.<br>
>><br>
>> Especially when working with browser triggered asynchronous requests,<br>
>> you can not rely on client-sided request synchronization.<br>
>> Weld, unlike OWB, grants a 1 second timeout prior to throwing a (the<br>
>> specified) BusyConversationException mitigating the effect a bit.<br>
>><br>
>> This is a rather strict un-configurable type of CC. Also its<br>
>> completely out of alignment with the other build-in scopes, offering no<br>
>> CC what so ever.<br>
>><br>
>> In the cases of Session- and Application-Scope, thread handling is left<br>
>> entirely to the developer, even so they are just as vulnerable in AJAX<br>
>> environments.<br>
>><br>
>> We should really consider introducing a common configurable mechanism,<br>
>> that is aligned across all scopes (obviously accounting for backwards<br>
>> compatibility in the case of conversation scope).<br>
>><br>
>> Would really appreciate some feedback.<br>
>><br>
>> Kind regards,<br>
>><br>
>> Stephan<br>
>><br>
>><br>
>><br>
>><br>
>> On Mon, 22 Feb 2016 at 23:10 Reza Rahman <<a href="mailto:Reza.Rahman@oracle.com" target="_blank">Reza.Rahman@oracle.com</a><br>
>> <mailto:<a href="mailto:Reza.Rahman@oracle.com" target="_blank">Reza.Rahman@oracle.com</a>>> wrote:<br>
>><br>
>> We've discussed this issue before. I definitely still think @Lock<br>
>> belongs in a modular CDI specification. It would be highly useful to<br>
>> both @Singleton and @ApplicationScoped. Today if I need to use<br>
>> declarative concurrency control for a shared component I am<br>
>> essentially forced to use EJB singleton - which shouldn't be the<br>
>> case and perhaps should not have been the case past Java EE 6.<br>
>><br>
>><br>
>>> On 2/19/2016 5:27 AM, Stephan Knitelius wrote:<br>
>>> Hi all,<br>
>>><br>
>>> CDI spec does not define a common concurrency control mechanism.<br>
>>> The time any type of concurrency control is mentioned is in<br>
>>> conjunction with EJB and a rather restrictive one for conversation<br>
>>> context.<br>
>>><br>
>>> CDI Spec:<br>
>>> The container ensures that a long-running conversation may be<br>
>>> associated with at most one request at a time, by blocking or<br>
>>> rejecting concurrent requests. If the container rejects a request,<br>
>>> it must associate the request with a new transient conversation<br>
>>> and throw an exception of<br>
>>> type|javax.enterprise.context.BusyConversationException|.<br>
>>><br>
>>><br>
>>> It would be helpful if a common configurable concurrency mechanism<br>
>>> (EJB Singleton style locking?) could be established for all normal<br>
>>> scopes.<br>
>>><br>
>>> What are your thoughts on this?<br>
>>><br>
>>> Regards,<br>
>>><br>
>>> Stephan<br>
>>><br>
>>><br>
>>><br>
>>><br>
>>><br>
>>><br>
>>><br>
>>><br>
>>><br>
>>><br>
>>> ______________________________________<br>
>>> *Stephan Knitelius*<br>
>>> Alteburger Str. 274<br>
>>> 50968 Köln / Cologne<br>
>>> Deutschland / Germany<br>
>>> <a href="mailto:stephan@knitelius.com" target="_blank">stephan@knitelius.com</a> <mailto:<a href="mailto:stephan@knitelius.com" target="_blank">stephan@knitelius.com</a>><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> <mailto:<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" rel="noreferrer" 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" rel="noreferrer" 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>
>><br>
>> _______________________________________________<br>
>> cdi-dev mailing list<br>
>> <a href="mailto:cdi-dev@lists.jboss.org" target="_blank">cdi-dev@lists.jboss.org</a> <mailto:<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" rel="noreferrer" 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<br>
>> the code under the Apache License, Version 2<br>
>> (<a href="http://www.apache.org/licenses/LICENSE-2.0.html" rel="noreferrer" target="_blank">http://www.apache.org/licenses/LICENSE-2.0.html</a>). For all other<br>
>> ideas provided on this list, the provider waives all patent and<br>
>> other intellectual 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" rel="noreferrer" 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" rel="noreferrer" 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>
><br>
> --<br>
> Martin Kouba<br>
> Software Engineer<br>
> Red Hat, Czech Republic<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" rel="noreferrer" 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" rel="noreferrer" 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>