OWB-702

2012-09-09 Thread Thomas Andraschko
Hi, i have already done some refactoring, fixed a bug because passivation removes the bag from the session and added unit tests for the whole failover stuff. I've sent the patch to mark for reviewing first because i didn't dare to create a issue + patch without reviewing :) Should i maybe

Re: OWB-702

2012-09-09 Thread Mark Struberg
yes please do and sorry that I had so less time lately :/ LieGrue, strub - Original Message - From: Thomas Andraschko zoi...@gmail.com To: dev@openwebbeans.apache.org Cc: Sent: Sunday, September 9, 2012 12:21 PM Subject: OWB-702 Hi, i have already done some refactoring,

Re: svn commit: r1363887 - /openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/logger/WebBeansLoggerFacade.java

2012-09-09 Thread Joseph Bergmark
I know was well over a month ago, but do you remember why: WebBeansUtil.getCurrentClassLoader().loadClass(factoryClassname) is problematic, but a direct call to: ClassLoader classloader = Thread.currentThread().getContextClassLoader(); is not? They appear basically the same. Unless I'm

Re: svn commit: r1363887 - /openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/logger/WebBeansLoggerFacade.java

2012-09-09 Thread Romain Manni-Bucau
Hi, Yeah it does exactly the same but the code is in a static block so it is executed when the class is loaded and the imports between both classes make it fail (circular dep). I didnt like to copy paste the code but it is a sure way to avoid linkageerror. About java 2 sec you are right it is