Return-Path: X-Original-To: apmail-incubator-deltaspike-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-deltaspike-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 6BF98D26B for ; Tue, 16 Oct 2012 06:19:49 +0000 (UTC) Received: (qmail 69070 invoked by uid 500); 16 Oct 2012 06:19:49 -0000 Delivered-To: apmail-incubator-deltaspike-dev-archive@incubator.apache.org Received: (qmail 68719 invoked by uid 500); 16 Oct 2012 06:19:42 -0000 Mailing-List: contact deltaspike-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: deltaspike-dev@incubator.apache.org Delivered-To: mailing list deltaspike-dev@incubator.apache.org Received: (qmail 68671 invoked by uid 99); 16 Oct 2012 06:19:40 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 16 Oct 2012 06:19:40 +0000 X-ASF-Spam-Status: No, hits=-5.0 required=5.0 tests=RCVD_IN_DNSWL_HI,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of jharting@redhat.com designates 209.132.183.28 as permitted sender) Received: from [209.132.183.28] (HELO mx1.redhat.com) (209.132.183.28) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 16 Oct 2012 06:19:34 +0000 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id q9G6JCj5020199 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Tue, 16 Oct 2012 02:19:12 -0400 Received: from [10.34.29.250] (dhcp-29-250.brq.redhat.com [10.34.29.250]) by int-mx09.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id q9G6JA45004589; Tue, 16 Oct 2012 02:19:11 -0400 Message-ID: <507CFC5E.1060603@redhat.com> Date: Tue, 16 Oct 2012 08:19:10 +0200 From: Jozef Hartinger User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120828 Thunderbird/15.0 MIME-Version: 1.0 To: Mark Struberg CC: "deltaspike-dev@incubator.apache.org" Subject: Re: seam-servlet stuff to deltaspike References: <1350054959.2604.YahooMailNeo@web28903.mail.ir2.yahoo.com> <507BCD70.3000302@redhat.com> <1350314566.23739.YahooMailNeo@web28903.mail.ir2.yahoo.com> In-Reply-To: <1350314566.23739.YahooMailNeo@web28903.mail.ir2.yahoo.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.68 on 10.5.11.22 X-Virus-Checked: Checked by ClamAV on apache.org #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 >> To: Jozef Hartinger >> 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 >> 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 >> wrote: >>>> This was one major outstanding issue. >>>>> >> https://issues.jboss.org/**browse/SOLDER-312 >>>>> On Sat, Oct 13, 2012 at 4:22 AM, Charles Moulliard >> >>>>> 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 >>>>>>> >>>>>>>> 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 >>>>>>>>> >>>>>>>>> +1 for modules/servlet :) >>>>>>>>>> LieGrue, >>>>>>>>>> strub >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> ----- Original Message ----- >>>>>>>>>> >>>>>>>>>>> From: Jason Porter >> >>>>>>>>>>> To: >> 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 >>>>>>>>>>> wrote: >>>>>>>>>>> >>>>>>>>>>> +1 >>>>>>>>>>>> *Romain Manni-Bucau* >>>>>>>>>>>> *Twitter: @rmannibucau >> >>>>>>>>>>>> >* >>>>>>>>>>>> *Blog: >> **http://rmannibucau.**wordpress.com/* >>>>>>>>>>>> < >>>>>>>>>>>> >> http://rmannibucau.wordpress.**com/ >>>>>>>>>>>> > >>>>>>>>>>>> *LinkedIn: >> **http://fr.linkedin.com/in/**rmannibucau* >>>>>>>>>>>> *Github: >> https://github.com/**rmannibucau* >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> 2012/10/12 Adrian Mitev >> >>>>>>>>>>>> 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 >>>>>>>> 2 - >>>>>>>>>>>>> >>>>>>>>>>>>> >> 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 >>>>>>>>>>> -- >>>>>>>>>>> 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 >>>>>>>>>>> >>>>>>>> -- >>>>>>>> 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 >>>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> 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 >>