Since WSRP work is almost impossible due to this issue in trunk, I've recreated the wsrp2-integration branch with the changes from revision 4242 reverted so that WSRP work can still progress. Hopefully, this won't lead to too many headaches come merging time… :( On Oct 1, 2010, at 7:20 PM, Christophe Laprun wrote: > Folks, > > I've spent the last couple of days trying to fix an issue with the WSRP admin UI which is currently behaving very erratically. After lots of trials and errors I have finally isolated the issue to commit 4242 in trunk: http://fisheye.jboss.org/changelog/gatein/portal/trunk?cs=4242 > > I have no idea why it's causing so many issues but I assume that something in that commit has broken the Portlet Bridge state management. Julien, Wesley: any ideas? > > Moving forward, I will make sure that we have some Selenium tests that check the WSRP admin and the gatein vs gatein WSRP consumption that are run in Hudson as we certainly could have caught that issue sooner (and avoided losing 2-3 days tracking the issue). > > Thanks. > > Cordialement / Best, > Chris > > == > Principal Software Engineer / JBoss Enterprise Middleware Red Hat, Inc. > Follow GateIn: http://blog.gatein.org / http://twitter.com/gatein > Follow me: http://metacosm.codepuccino.com / http://twitter.com/metacosm > > > _______________________________________________ > gatein-dev mailing list > gatein-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/gatein-dev Cordialement / Best, Chris == Principal Software Engineer / JBoss Enterprise Middleware Red Hat, Inc. Follow GateIn: http://blog.gatein.org / http://twitter.com/gatein Follow me: http://metacosm.codepuccino.com / http://twitter.com/metacosm _______________________________________________ gatein-dev mailing list gatein-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/gatein-dev

could we have Wesley to look at this problem as he seems the most qualified to understand the problem ?

On Oct 1, 2010, at 7:20 PM, Christophe Laprun wrote: > Folks, > > I've spent the last couple of days trying to fix an issue with the WSRP admin UI which is currently behaving very erratically. After lots of trials and errors I have finally isolated the issue to commit 4242 in trunk: http://fisheye.jboss.org/changelog/gatein/portal/trunk?cs=4242 > > I have no idea why it's causing so many issues but I assume that something in that commit has broken the Portlet Bridge state management. Julien, Wesley: any ideas? > > Moving forward, I will make sure that we have some Selenium tests that check the WSRP admin and the gatein vs gatein WSRP consumption that are run in Hudson as we certainly could have caught that issue sooner (and avoided losing 2-3 days tracking the issue). > > Thanks. > > Cordialement / Best, > Chris > > == > Principal Software Engineer / JBoss Enterprise Middleware Red Hat, Inc. > Follow GateIn: http://blog.gatein.org / http://twitter.com/gatein > Follow me: http://metacosm.codepuccino.com / http://twitter.com/metacosm > > > _______________________________________________ > gatein-dev mailing list > gatein-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/gatein-dev

can you reproduce this issue with a simpler bridge portlet ? on my side I can try to give you a patch that fix the issue I did but without doing a refactor that is inside (moving the computing of the supports map in the getter) On Oct 2, 2010, at 4:11 PM, Christophe Laprun wrote: > On Oct 2, 2010, at 3:57 PM, Julien Viet wrote: > > >> could we have Wesley to look at this problem as he seems the most qualified to understand the problem ? >> > It's probably going to require collaboration because I don't think the issue is solely in PBR or GateIn… > > >> On Oct 1, 2010, at 7:20 PM, Christophe Laprun wrote: >> >> >>> Folks, >>> >>> I've spent the last couple of days trying to fix an issue with the WSRP admin UI which is currently behaving very erratically. After lots of trials and errors I have finally isolated the issue to commit 4242 in trunk: http://fisheye.jboss.org/changelog/gatein/portal/trunk?cs=4242 >>> >>> I have no idea why it's causing so many issues but I assume that something in that commit has broken the Portlet Bridge state management. Julien, Wesley: any ideas? >>> >>> Moving forward, I will make sure that we have some Selenium tests that check the WSRP admin and the gatein vs gatein WSRP consumption that are run in Hudson as we certainly could have caught that issue sooner (and avoided losing 2-3 days tracking the issue). >>> >>> Thanks. >>> >>> Cordialement / Best, >>> Chris >>> >>> == >>> Principal Software Engineer / JBoss Enterprise Middleware Red Hat, Inc. >>> Follow GateIn: http://blog.gatein.org / http://twitter.com/gatein >>> Follow me: http://metacosm.codepuccino.com / http://twitter.com/metacosm >>> >>> >>> _______________________________________________ >>> gatein-dev mailing list >>> gatein-dev(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/gatein-dev >>> >> > Cordialement / Best, > Chris > > == > Principal Software Engineer / JBoss Enterprise Middleware Red Hat, Inc. > Follow GateIn: http://blog.gatein.org / http://twitter.com/gatein > Follow me: http://metacosm.codepuccino.com / http://twitter.com/metacosm > > _______________________________________________ gatein-dev mailing list gatein-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/gatein-dev

so finally problem was solved which is the most important thing, but there is something weird that I think only Wesley our bridge expert could find out. On Oct 4, 2010, at 1:18 PM, Thomas Heute wrote: > Adding Wesley (just in case he's not on this ML) > > On 10/02/2010 05:49 PM, Julien Viet wrote: >> can you reproduce this issue with a simpler bridge portlet ? >> >> on my side I can try to give you a patch that fix the issue I did but without doing a refactor that is inside (moving the computing of the supports map in the getter) >> >> On Oct 2, 2010, at 4:11 PM, Christophe Laprun wrote: >> >> >>> On Oct 2, 2010, at 3:57 PM, Julien Viet wrote: >>> >>> >>>> could we have Wesley to look at this problem as he seems the most qualified to understand the problem ? >>>> >>> It's probably going to require collaboration because I don't think the issue is solely in PBR or GateIn… >>> >>> >>>> On Oct 1, 2010, at 7:20 PM, Christophe Laprun wrote: >>>> >>>> >>>>> Folks, >>>>> >>>>> I've spent the last couple of days trying to fix an issue with the WSRP admin UI which is currently behaving very erratically. After lots of trials and errors I have finally isolated the issue to commit 4242 in trunk: http://fisheye.jboss.org/changelog/gatein/portal/trunk?cs=4242 >>>>> >>>>> I have no idea why it's causing so many issues but I assume that something in that commit has broken the Portlet Bridge state management. Julien, Wesley: any ideas? >>>>> >>>>> Moving forward, I will make sure that we have some Selenium tests that check the WSRP admin and the gatein vs gatein WSRP consumption that are run in Hudson as we certainly could have caught that issue sooner (and avoided losing 2-3 days tracking the issue). >>>>> >>>>> Thanks. >>>>> >>>>> Cordialement / Best, >>>>> Chris >>>>> >>>>> == >>>>> Principal Software Engineer / JBoss Enterprise Middleware Red Hat, Inc. >>>>> Follow GateIn: http://blog.gatein.org / http://twitter.com/gatein >>>>> Follow me: http://metacosm.codepuccino.com / http://twitter.com/metacosm >>>>> >>>>> >>>>> _______________________________________________ >>>>> gatein-dev mailing list >>>>> gatein-dev(a)lists.jboss.org >>>>> https://lists.jboss.org/mailman/listinfo/gatein-dev >>>>> >>>> >>> Cordialement / Best, >>> Chris >>> >>> == >>> Principal Software Engineer / JBoss Enterprise Middleware Red Hat, Inc. >>> Follow GateIn: http://blog.gatein.org / http://twitter.com/gatein >>> Follow me: http://metacosm.codepuccino.com / http://twitter.com/metacosm >>> >>> >> >> _______________________________________________ >> gatein-dev mailing list >> gatein-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/gatein-dev >> >