On 12/12/2007, Alin Dreghiciu <[EMAIL PROTECTED]> wrote:
>
> Excellent. So, I guess that this is a good indicator that we should
> release 0.2.2.


++1

Alin
>
> PS> Then I can throw everything away and start from scratch :)).


Pax-WebNG ?

On Dec 12, 2007 1:58 PM, Stuart McCulloch <[EMAIL PROTECTED]>
> wrote:
> > On 12/12/2007, Alin Dreghiciu <[EMAIL PROTECTED]> wrote:
> >
> > > I committed the changes. It should work now :)
> >
> > woohoo - TCK passes!
> >
> >
> > > On Dec 12, 2007 1:06 PM, Alin Dreghiciu <[EMAIL PROTECTED]> wrote:
> > > > I was already working on this. For the exception bellow I already
> have
> > > > a fix. Is a mater of couple of hours till I commit it.
> > > > Pretty close now :)
> > > >
> > > >
> > > > On Dec 12, 2007 12:47 PM, Stuart McCulloch <
> [EMAIL PROTECTED]>
> > wrote:
> > > > > On 12/12/2007, Stuart McCulloch < [EMAIL PROTECTED]>
> wrote:
> > > > >
> > > > > > On 12/12/2007, Stuart McCulloch <[EMAIL PROTECTED] >
> > wrote:
> > > > > >
> > > > > > > On 12/12/2007, Alin Dreghiciu <[EMAIL PROTECTED]> wrote:
> > > > > > >
> > > > > > > > Thanx Stuart,
> > > > > > > >
> > > > > > > > Can you figure out what's the test about. From the exception
> I
> > do not
> > > > > > > > know what they expect.
> > > > > > >
> > > > > > >
> > > > > > > basically the test registers one servlet under a HttpContext,
> > > > > unregisters it,
> > > > > > > registers a different servlet, and then tries to access a
> resource
> > from
> > > > > the
> > > > > > > second servlet - it fails because it can't read the second
> > resource...
> > > > > HTH
> > > > > >
> > > > > >
> > > > > > FYI, the issue is with the reusing of contexts and registrations
> in
> > the
> > > > > new code
> > > > > > - there's already an HttpServiceContext in the map, but the new
> > > > > registrations
> > > > > > aren't merged in with the cached HttpServiceContext
> > > > >
> > > > > stopping the old context from being removed when there are no
> > registrations
> > > > > (in StartedHttpService.unregister) works around this issue, but
> then
> > another
> > > > > new exception appears, which may also be related:
> > > > >
> > > > >      [java] Log: During execution of BundleStopping
> > > > > org.osgi.framework.BundleException : Exception in
> > > > > org.osgi.test.cases.http.tbc.tb1.HttpTestBundle2.start() of bundle
> 30.
> > > > >      [java] org.osgi.framework.BundleException: Exception in
> > > > > org.osgi.test.cases.http.tbc.tb1.HttpTestBundle2.start() of bundle
> 30.
> > > > >       [java]     at
> > > > >
> >
> org.eclipse.osgi.framework.internal.core.BundleContextImpl.startActivator
> > (BundleContextImpl.java:1018)
> > > > >      [java]     at
> > > > >
> > org.eclipse.osgi.framework.internal.core.BundleContextImpl.start(
> BundleContextImpl.java
> > > > > :974)
> > > > >      [java]     at
> > > > >
> > org.eclipse.osgi.framework.internal.core.BundleHost.startWorker(
> BundleHost.java:346)
> > > > >      [java]     at
> > > > >
> > org.eclipse.osgi.framework.internal.core.AbstractBundle.start(
> AbstractBundle.java:260)
> > > > >      [java]     at
> > > > >
> > org.eclipse.osgi.framework.internal.core.AbstractBundle.start(
> AbstractBundle.java:252)
> > > > >      [java]     at
> > > > >
> > org.osgi.test.cases.http.tbc.HttpTestBundle1.BundleStopping(
> HttpTestBundle1.java
> > :284)
> > > > >
> > > > >       [java]     at
> > sun.reflect.NativeMethodAccessorImpl.invoke0(Native
> > > > > Method)
> > > > >      [java]     at
> > > > >
> > sun.reflect.NativeMethodAccessorImpl.invoke(
> NativeMethodAccessorImpl.java
> > :64)
> > > > >      [java]     at sun.reflect.DelegatingMethodAccessorImpl.invoke
> > > > > (DelegatingMethodAccessorImpl.java:43)
> > > > >      [java]     at java.lang.reflect.Method.invoke(Method.java
> :615)
> > > > >      [java]     at
> > > > >
> > org.osgi.test.cases.util.DefaultTestBundleControl.run(
> DefaultTestBundleControl.java:271)
> > > > >       [java]     at java.lang.Thread.run(Thread.java:801)
> > > > >      [java] Caused by: org.osgi.service.http.NamespaceException:
> alias
> > is
> > > > > already in use
> > > > >      [java]     at
> > > > > org.ops4j.pax.web.service.internal.RegistrationsImpl.validateAlias
> > > > > (RegistrationsImpl.java :166)
> > > > >      [java]     at
> > > > >
> >
> org.ops4j.pax.web.service.internal.RegistrationsImpl.validateRegisterServletArguments
> (RegistrationsImpl.java:125)
> > > > >      [java]     at
> > > > >
> org.ops4j.pax.web.service.internal.RegistrationsImpl.registerServlet
> > > > > (RegistrationsImpl.java:67)
> > > > >      [java]     at
> > > > >
> > org.ops4j.pax.web.service.internal.StartedHttpService.registerServlet(
> StartedHttpService.java:101)
> > > > >      [java]     at
> > > > >
> org.ops4j.pax.web.service.internal.HttpServiceProxy.registerServlet
> > > > > (HttpServiceProxy.java:50)
> > > > >      [java]     at
> > > > >
> > org.osgi.test.cases.http.tbc.tb1.HttpTestBundle2.start(
> HttpTestBundle2.java
> > :30)
> > > > >      [java]     at
> > > > >
> > org.eclipse.osgi.framework.internal.core.BundleContextImpl$2.run(
> BundleContextImpl.java
> > > > > :999)
> > > > >      [java]     at
> > > > > java.security.AccessController.doPrivileged
> > (AccessController.java:242)
> > > > >      [java]     at
> > > > >
> >
> org.eclipse.osgi.framework.internal.core.BundleContextImpl.startActivator(
> BundleContextImpl.java:993)
> > > > >      [java]     ... 11 more
> > > > >
> > > > >
> > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > > Alin
> > > > > > > >
> > > > > > > > On Dec 12, 2007 5:16 AM, Stuart McCulloch
> > > > > <[EMAIL PROTECTED]> wrote:
> > > > > > > > > On 12/12/2007, Alin Dreghiciu < [EMAIL PROTECTED] >
> wrote:
> > > > > > > > >
> > > > > > > > > > Hi,
> > > > > > > > > >
> > > > > > > > > > I committed quite a big change required by PAXWEB-21 and
> > > > > PAXWEB-55,
> > > > > > > > > > related to isolation of servlet contexts per http
> context.
> > > > > > > > > > My tests were successful but I did not had a full blown
> app
> > to
> > > > > test with.
> > > > > > > > > >
> > > > > > > > > > Stuart, can you run the TCK?
> > > > > > > > >
> > > > > > > > > I just fixed a NPE that was blocking the TCK, but now get
> a
> > new
> > > > > > > > > FileNotFoundException:
> > > > > > > > >
> > > > > > > > >      [java] Log: During execution of
> > SimpleServletUnregistration
> > > > > > > > > java.io.FileNotFoundException :
> > > > > http://localhost:8083/tc2servlet?TestCase=2
> > > > > > > > >      [java] java.io.FileNotFoundException :
> > > > > > > > > http://localhost:8083/tc2servlet?TestCase=2
> > > > > > > > >      [java]     at
> > > > > > > > > sun.net.www.protocol.http.HttpURLConnection.getInputStream
> > > > > (HttpURLConnection.java:1177)
> > > > > > > > >      [java]     at java.net.URL.openStream(URL.java:1041)
> > > > > > > > >      [java]     at
> > > > > > > > >
> > > > >
> > org.osgi.test.cases.http.tbc.HttpTestBundle1.SimpleServletUnregistration
> > > > > > > > > (HttpTestBundle1.java :130)
> > > > > > > > >      [java]     at
> > > > > sun.reflect.NativeMethodAccessorImpl.invoke0(Native
> > > > > > > > > Method)
> > > > > > > > >      [java]     at
> > > > > > > > >
> > > > > sun.reflect.NativeMethodAccessorImpl.invoke
> > (NativeMethodAccessorImpl.java:64)
> > > > > > > > >      [java]     at
> > > > > > > > >
> > > > >
> > sun.reflect.DelegatingMethodAccessorImpl.invoke(
> DelegatingMethodAccessorImpl.java:43)
> > > > > > > > >      [java]     at
> > java.lang.reflect.Method.invoke(Method.java:615)
> > > > > > > > >      [java]     at
> > > > > org.osgi.test.cases.util.DefaultTestBundleControl.run
> > > > > > > > > ( DefaultTestBundleControl.java:271)
> > > > > > > > >      [java]     at java.lang.Thread.run(Thread.java:801)
> > > > > > > > >
> > > > > > > > > this didn't happen before, so it's probably related to the
> > recent
> > > > > context
> > > > > > > > > isolation change...
> > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > There are still some issues related to stopping bundles
> but
> > I will
> > > > > > > > > > address them asap.
> > > > > > > > > >
> > > > > > > > > > Looking at the current overall implementation of Pax Web
> > that I
> > > > > do
> > > > > > > > > > not like to much + some research I had done lately how
> could
> > be
> > > > > done
> > > > > > > > > > better I guess that I will son start to rewrite it from
> > scratch,
> > > > > this
> > > > > > > > > > after we will stabilize the current version.
> > > > > > > > > >
> > > > > > > > > > For those that used the Event Listeners I changed the
> > signature of
> > > > > > > > > > registering a listener to add the http context the
> listener
> > will
> > > > > > > > > > listen to, so please take a look.
> > > > > > > > > >
> > > > > > > > > > Alin
> > > > > > > > > >
> > > > > > > > > > _______________________________________________
> > > > > > > > > > general mailing list
> > > > > > > > > > [email protected]
> > > > > > > > > > http://lists.ops4j.org/mailman/listinfo/general
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > --
> > > > > > > > > Cheers, Stuart
> > > > > > > > > _______________________________________________
> > > > > > > > > general mailing list
> > > > > > > > > [email protected]
> > > > > > > > > http://lists.ops4j.org/mailman/listinfo/general
> > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > > _______________________________________________
> > > > > > > > general mailing list
> > > > > > > > [email protected]
> > > > > > > > http://lists.ops4j.org/mailman/listinfo/general
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Cheers, Stuart
> > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Cheers, Stuart
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Cheers, Stuart
> > > > > _______________________________________________
> > > > > general mailing list
> > > > > [email protected]
> > > > > http://lists.ops4j.org/mailman/listinfo/general
> > > > >
> > > > >
> > > >
> > >
> > > _______________________________________________
> > > general mailing list
> > > [email protected]
> > > http://lists.ops4j.org/mailman/listinfo/general
> > >
> >
> >
> >
> > --
> > Cheers, Stuart
> > _______________________________________________
> > general mailing list
> > [email protected]
> > http://lists.ops4j.org/mailman/listinfo/general
> >
> >
>
> _______________________________________________
> general mailing list
> [email protected]
> http://lists.ops4j.org/mailman/listinfo/general
>



-- 
Cheers, Stuart
_______________________________________________
general mailing list
[email protected]
http://lists.ops4j.org/mailman/listinfo/general

Reply via email to