On Mon, 2012-08-06 at 17:12 +0200, Julien Viet wrote: > On Aug 6, 2012, at 5:01 PM, Matt Wringe <mwringe(a)redhat.com&gt; wrote: > >> On Sat, 2012-08-04 at 10:47 +0200, Julien Viet wrote: >>> >>> Julien Viet >>> julienviet.com >>> >>> >>> On Saturday, August 4, 2012 at 4:01 AM, Matt Wringe wrote: >>> >>>> On Sat, 2012-08-04 at 00:11 +0200, Julien Viet wrote: >>>>> do you mean that bolek tag 3.4.0.M01 is not correct and we need to do it after your PR ? >>>> >>>> >>>> The device property based redirects currently don't work properly (well, >>>> not at all right now), user-agent redirects do work properly which are >>>> more important. Not sure if this is really important enough to do >>>> another tag. >>>> >>>> >>> >>> >>> ok >>>> >>>> I would have done a PR by now, but my tests are failing due to issues >>>> with the kernelbootstrap is not properly getting disposed and eating all >>>> the memory (also, why are we still using such an old version of junit >>>> without support for @beforeclass and @afterclass?) >>>> >>>> >> >> Is there anything special I need to do to release the resources used by >> the kernelbootstrap? Calling bootstrap.dispose does not release the >> resources and eventually leads to the heap space running out of memory. > > the kernel bootstrap is a nice facade in front of a pile of crap : had to use some hack to shut down things. > > can you identify the leak ? > > >> >> Since I can't use @beforeclass or @afterclass (to only create the >> kernelbootstrap before the suite and dispose of it after). I will have >> to recreate the bootstrap for each test and since the bootstrap doesn't >> release its resources, it cause the running out of memory before the >> tests are completed. > > can you identify the leak with JProfiler or GC verbosity ? A quick look at the heap dump shows that its most likely a problem with jcr/jboss cache not being released (org.jboss.cache.lock.MVCCLockManager specifically). But I am not really sure if I am looking at the results correctly in the eclipse memory analyzer. I updated the tests to get around this issue by only having junit call one method for the tests instead of multiple methods (so I can create the bootstrap at the start of this method and dispose of it at the end, and not have to do this for each test method). > actually I whish we can upgrade to JUnit 4, as we could use @RunWith in some places (looking at Byteman). > > however those classes are used by some UXP team (at least social) as basis for their unit testing, so we should ensure that stuff will remain as compatible as possible. > > >> >>> because it was working fine until now and it could take time to migrate unit tests. >>> >>>> >>>>> Julien Viet >>>>> julienviet.com (http://julienviet.com) >>>>> >>>>> >>>>> On Friday, August 3, 2012 at 11:39 PM, Matt Wringe wrote: >>>>> >>>>>> On Thu, 2012-08-02 at 20:39 +0200, Boleslaw Dawidowicz wrote: >>>>>>> Here is tag: >>>>>>> https://github.com/gatein/gatein-portal/tree/3.4.0.M01 >>>>>>> >>>>>>> Here are download links: >>>>>>> http://www.jboss.org/gatein/downloads/gateinportal >>>>>>> >>>>>>> Here is documentation: >>>>>>> http://www.jboss.org/gatein/documentation/gateinportal >>>>>>> >>>>>>> as always I would be grateful if everyone try out links above and report problems. >>>>>>> >>>>>>> At this point we need blog posts from Matt about mobile and from Julien about Javascript improvements. It doesn't make sense to announce software on Friday so we will probably push the news on Monday. Julien will handle it. >>>>>> >>>>>> Just a quick update: I have a blog post and docs mostly ready for the >>>>>> mobile stuff. I unfortunately ran into a couple of minor issues with the >>>>>> mobile stuff while writing the docs (double checking that everything was >>>>>> working as expected). Will post the docs/blog and do a pull request >>>>>> either later tonight or tomorrow with the fixes and updated tests. >>>>>> >>>>>>> I will be mostly disconnected next two weeks. In case we need to repackage things again Trong has all needed permissions and proper knowledge. Thomas can help with Magnolia on jboss.org (http://jboss.org). >>>>>>> >>>>>>> Thanks everyone for hard work and help with this release! >>>>>>> >>>>>>> Bolek >>>>>>> >>>>>>> >>>>>>> >>>>>>> _______________________________________________ >>>>>>> gatein-dev mailing list >>>>>>> gatein-dev(a)lists.jboss.org (mailto:gatein-dev@lists.jboss.org) >>>>>>> https://lists.jboss.org/mailman/listinfo/gatein-dev >>>>>>> >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> _______________________________________________ >>>>>> gatein-dev mailing list >>>>>> gatein-dev(a)lists.jboss.org (mailto:gatein-dev@lists.jboss.org) >>>>>> https://lists.jboss.org/mailman/listinfo/gatein-dev >>>>>> >>>>> >>>>> >>>> >>>> >>>> >>> >>> >> >> >