I don't understand why those classes were duplicated, most probably a mistake. If you move TenantLazyLoaderValve into tomcat.ext you may have to move many other classes from webapp-mgt too. Better to leave it as it is and fix the root cause.
On Mon, Mar 19, 2012 at 6:36 PM, Kishanthan Thangarajah <[email protected] > wrote: > Hi, > I find that CarbonTomcatValve and TomcatValveContainer are duplicated in > web-app mgt. They are already available in tomcat.ext bundle and > CompositeValve(which is registered with tomcat) is using the > TomcatValveContainer found in tomcat.ext bundle. This will cause a problem > in webapp-mgt with TenantLazyLoaderValve. Because webapp-mgt service > component is using TomcatValveContainer found in web-app component. So > the TenantLazyLoaderValve will never be called by tomcat. These duplicated > classes should be removed form webapp-mgt bundle. > Also shall we move TenantLazyLoaderValve to tomcat.ext? > > Thanks, > Kishanthan. > > -- > *Kishanthan Thangarajah* > Software Engineer, > Development Technologies Team, > WSO2, Inc. > lean.enterprise.middleware > > Mobile - +94773426635 > Blog - *http://kishanthan.wordpress.com* > Twitter - *http://twitter.com/kishanthan* > > > _______________________________________________ > Dev mailing list > [email protected] > http://wso2.org/cgi-bin/mailman/listinfo/dev > > -- *Afkham Azeez* Director of Architecture; WSO2, Inc.; http://wso2.com Member; Apache Software Foundation; http://www.apache.org/ * <http://www.apache.org/>** email: **[email protected]* <[email protected]>* cell: +94 77 3320919 blog: **http://blog.afkham.org* <http://blog.afkham.org>* twitter: **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> * linked-in: **http://lk.linkedin.com/in/afkhamazeez* * * *Lean . Enterprise . Middleware*
_______________________________________________ Dev mailing list [email protected] http://wso2.org/cgi-bin/mailman/listinfo/dev
