incubator-deltaspike-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Struberg <strub...@yahoo.de>
Subject Re: seam-servlet stuff to deltaspike
Date Tue, 16 Oct 2012 08:48:25 GMT
Another argument for interpreting @ApplicationScoped as web-application singleton like suggested
in  CDI-129.
I f****n care what some containers got wrong by taking it as 1 per EAR.

I now talked with

* serlvet EG members
* Ed, JSF spec lead
* Spring folks
* tons of user
* even you JBoss Seam guys

ALL of them AND THE CDI SPEC (see 2.4.1 "The @RequestScoped, @ApplicationScoped and @SessionScoped
annotations defined in Section 6.7, “Context management for built-in scopes” represent
the standard scopes defined by the Java Servlets specification.") interpret @ApplicationScoped
as 1 per webapp.

damn, I really f***n care what some containers did wrong so far (including our own)! All what
is important is to fix the behaviour in the future. It's also that ALL CDI Extensions expect
an own BeanManager per WebApplication. That would be perfectly broken now as well and cause
lots of non-portability.


 LieGrue,
strub



----- Original Message -----
> From: Jozef Hartinger <jharting@redhat.com>
> To: Mark Struberg <struberg@yahoo.de>
> Cc: "deltaspike-dev@incubator.apache.org" <deltaspike-dev@incubator.apache.org>
> Sent: Tuesday, October 16, 2012 8:19 AM
> Subject: Re: seam-servlet stuff to deltaspike
> 
> #2 could be split into two issues:
> 
> 2a) Injection of Servlet artefacts
> 
> Solder stores ServletContext in an @ApplicationScoped holder which 
> caused a clash between multiple ServletContexts in a multiwar ear 
> deployment. This can be solved easily by using something other than 
> @ApplicationScoped holder for holding the reference.
> 
> 2b) Lifecycle events
> 
> Solder propagates servlet lifecyce events e.g. @Initialized 
> ServletContext. In a multi-war ear deployment an event with payload that 
> represents a servlet context of war1 is fired to all matching observer 
> methods including those in different wars which may be confusing.
> 
> We got this right in Weld but we were able to do that because we have 
> much more information about a deployment structure compared what a CDI 
> extension has. I am not sure if this can be implemented properly as a 
> CDI extension.
> 
> On 10/15/2012 05:22 PM, Mark Struberg wrote:
>>  what was the problem actually?
>> 
>>  LieGrue,
>>  strub
>> 
>> 
>> 
>> 
>>  ----- Original Message -----
>>>  From: Jason Porter <lightguard.jp@gmail.com>
>>>  To: Jozef Hartinger <jharting@redhat.com>
>>>  Cc: deltaspike-dev@incubator.apache.org
>>>  Sent: Monday, October 15, 2012 5:19 PM
>>>  Subject: Re: seam-servlet stuff to deltaspike
>>> 
>>>  No problem at all with #1, #2 is a bit difficult to solve. Jozef, have 
> you
>>>  solved this in Weld 2.0? If so, how do you propose we solve it in DS?
>>> 
>>>  On Mon, Oct 15, 2012 at 2:46 AM, Jozef Hartinger
>>>  <jharting@redhat.com>wrote:
>>> 
>>>>    There are two issues I am aware of:
>>>> 
>>>>    1) The injectable Servlet artifacts should define a 
> deltaspike-specific
>>>>    qualifier in order to prevent conflict with CDI 1.1 which defines 
> these
>>>>    artifacts in the @Default space.
>>>> 
>>>>    2) There was an issue in solder related to multi-war ear 
> deployment which
>>>>    is hard to get right
>>>> 
>>>> 
>>>>    On 10/13/2012 07:39 PM, Jason Porter wrote:
>>>> 
>>>>>    Were there other issues? That one is easy to fix. I thought 
> there was
>>>>>    something with the producers  at some point.
>>>>> 
>>>>>    Sent from my iPhone
>>>>> 
>>>>>    On Oct 13, 2012, at 11:17, Cody Lerum 
> <cody.lerum@gmail.com>
>>>  wrote:
>>>>>     This was one major outstanding issue.
>>>>>> 
>>> 
> https://issues.jboss.org/**browse/SOLDER-312<https://issues.jboss.org/browse/SOLDER-312>
>>>>>>    On Sat, Oct 13, 2012 at 4:22 AM, Charles Moulliard
>>>  <ch007m@gmail.com>
>>>>>>    wrote:
>>>>>> 
>>>>>>>    +1
>>>>>>> 
>>>>>>>    On Sat, Oct 13, 2012 at 10:56 AM, Christian Kaltepoth 
> <
>>>>>>>   christian@kaltepoth.de> wrote:
>>>>>>> 
>>>>>>>     +1 for adding it to 0.4 as a separate servlet 
> module.
>>>>>>>>    I think these are very important features. 
> Especially the
>>>  event
>>>>>>>>    propagation and the injection of servlet-related 
> objects.
>>>>>>>> 
>>>>>>>>    Christian
>>>>>>>> 
>>>>>>>>    2012/10/12 Jason Porter 
> <lightguard.jp@gmail.com>
>>>>>>>> 
>>>>>>>>>    Sounds like we're good to add it. Shall 
> we add it
>>>  for v0.4?
>>>>>>>>>    On Fri, Oct 12, 2012 at 11:04 AM, Gerhard 
> Petracek <
>>>>>>>>>   gerhard.petracek@gmail.com> wrote:
>>>>>>>>> 
>>>>>>>>>     +1 for an own module.
>>>>>>>>>>    regards,
>>>>>>>>>>    gerhard
>>>>>>>>>> 
>>>>>>>>>> 
>>>>>>>>>> 
>>>>>>>>>>    2012/10/12 Mark Struberg 
> <struberg@yahoo.de>
>>>>>>>>>> 
>>>>>>>>>>     +1 for modules/servlet :)
>>>>>>>>>>>    LieGrue,
>>>>>>>>>>>    strub
>>>>>>>>>>> 
>>>>>>>>>>> 
>>>>>>>>>>> 
>>>>>>>>>>> 
>>>>>>>>>>>    ----- Original Message -----
>>>>>>>>>>> 
>>>>>>>>>>>>    From: Jason Porter
>>>  <lightguard.jp@gmail.com>
>>>>>>>>>>>>    To:
>>> 
> deltaspike-dev@incubator.**apache.org<deltaspike-dev@incubator.apache.org>
>>>>>>>>>>>>    Cc:
>>>>>>>>>>>>    Sent: Friday, October 12, 2012 
> 5:12 PM
>>>>>>>>>>>>    Subject: Re: seam-servlet stuff 
> to
>>>  deltaspike
>>>>>>>>>>>>    I have no problem adding it. It 
> certainly
>>>  should be its own module
>>>>>>>>>>>    though.
>>>>>>>>>>> 
>>>>>>>>>>>>    We may also need to rethink some 
> of how the
>>>  code was working. I
>>>>>>>>>>>    remember
>>>>>>>>>>>    there being problems, but maybe 
> it's simply
>>>  because we put it into
>>>>>>>>>>>    solder.
>>>>>>>>>>> 
>>>>>>>>>>>>    On Fri, Oct 12, 2012 at 9:08 AM, 
> Romain
>>>  Manni-Bucau
>>>>>>>>>>>>   
> <rmannibucau@gmail.com>wrote:
>>>>>>>>>>>> 
>>>>>>>>>>>>     +1
>>>>>>>>>>>>>    *Romain Manni-Bucau*
>>>>>>>>>>>>>    *Twitter: @rmannibucau
>>> 
> <https://twitter.com/**rmannibucau<https://twitter.com/rmannibucau>
>>>>>>>>>>>>>    >*
>>>>>>>>>>>>>    *Blog:
>>> 
> **http://rmannibucau.**wordpress.com/*<http://rmannibucau.wordpress.com/*>
>>>>>>>>>>>>>    <
>>>>>>>>>>>>> 
>>> 
> http://rmannibucau.wordpress.**com/<http://rmannibucau.wordpress.com/>
>>>>>>>>>>>>>    >
>>>>>>>>>>>>>    *LinkedIn:
>>> 
> **http://fr.linkedin.com/in/**rmannibucau*<http://fr.linkedin.com/in/rmannibucau*>
>>>>>>>>>>>>>    *Github:
>>> 
> https://github.com/**rmannibucau*<https://github.com/rmannibucau*>
>>>>>>>>>>>>> 
>>>>>>>>>>>>> 
>>>>>>>>>>>>> 
>>>>>>>>>>>>>    2012/10/12 Adrian Mitev
>>>  <adrian.mitev@gmail.com>
>>>>>>>>>>>>>     Hi all! The stuff in the old
>>>  seam-servlet module [1], [2] and
>>>>>>>>>>>>>    [3]
>>>>>>>>>    (now
>>>>>>>>>>>>    merged in seam-solder) are quite 
> useful and
>>>  are great
>>>>>>>>>>>>>    candidate for
>>>>>>>>>     adding
>>>>>>>>>>>>>>    in Deltaspike.
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>    1 -
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>     
>>>  http://docs.jboss.org/seam/3/**3.1.0.Final/reference/en-US/**
>>> 
> html/servlet-events.html<http://docs.jboss.org/seam/3/3.1.0.Final/reference/en-US/html/servlet-events.html>
>>>>>>>>>     2 -
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>     
>>>  http://docs.jboss.org/seam/3/**3.1.0.Final/reference/en-US/**
>>> 
> html/injectablerefs.html<http://docs.jboss.org/seam/3/3.1.0.Final/reference/en-US/html/injectablerefs.html>
>>>>>>>>>     3 -
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>     
>>>  http://docs.jboss.org/seam/3/**3.1.0.Final/reference/en-US/**
>>> 
> html/exception-handling.html<http://docs.jboss.org/seam/3/3.1.0.Final/reference/en-US/html/exception-handling.html>
>>>>>>>>>>>>    --
>>>>>>>>>>>>    Jason Porter
>>>>>>>>>>>> 
>>> 
> http://lightguard-jp.blogspot.**com<http://lightguard-jp.blogspot.com>
>>> 
> http://twitter.com/**lightguardjp<http://twitter.com/lightguardjp>
>>>>>>>>>>>>    Software Engineer
>>>>>>>>>>>>    Open Source Advocate
>>>>>>>>>>>>    Author of Seam Catch - Next 
> Generation Java
>>>  Exception Handling
>>>>>>>>>>>>    PGP key id: 926CCFF5
>>>>>>>>>>>>    PGP key available at: 
> keyserver.net,
>>>  pgp.mit.edu
>>>>>>>>>>>> 
>>>>>>>>>    --
>>>>>>>>>    Jason Porter
>>>>>>>>> 
>>> 
> http://lightguard-jp.blogspot.**com<http://lightguard-jp.blogspot.com>
>>>>>>>>>   http://twitter.com/**lightguardjp
>>>  <http://twitter.com/lightguardjp>
>>>>>>>>>    Software Engineer
>>>>>>>>>    Open Source Advocate
>>>>>>>>>    Author of Seam Catch - Next Generation Java 
> Exception
>>>  Handling
>>>>>>>>>    PGP key id: 926CCFF5
>>>>>>>>>    PGP key available at: keyserver.net, 
> pgp.mit.edu
>>>>>>>>> 
>>>>>>>> 
>>>>>>>> 
>>>>>>>>    --
>>>>>>>>    Christian Kaltepoth
>>>>>>>>    Blog: http://chkal.blogspot.com/
>>>>>>>>    Twitter: http://twitter.com/chkal
>>>>>>>> 
>>>>>>>> 
>>>>>>>    --
>>>>>>>    Charles Moulliard
>>>>>>>    Apache Committer / Sr. Enterprise Architect (RedHat)
>>>>>>>    Twitter : @cmoulliard | Blog : 
> http://cmoulliard.blogspot.com
>>>>>>> 
>>> 
>>>  -- 
>>>  Jason Porter
>>>  http://lightguard-jp.blogspot.com
>>>  http://twitter.com/lightguardjp
>>> 
>>>  Software Engineer
>>>  Open Source Advocate
>>>  Author of Seam Catch - Next Generation Java Exception Handling
>>> 
>>>  PGP key id: 926CCFF5
>>>  PGP key available at: keyserver.net, pgp.mit.edu
>>> 
> 

Mime
View raw message