The fundamental problem with the old API is that it exposed the internal representation, which was called CarbonContextHolder. That is what made it messy & difficult to understand. With the new API, you have CarbonContext & PrivilegedCarbonContext. The internal representation, CarbonContextDataHolder, is exposed only via these two clean APIs. The dangers of this hard to understand API were apparent when people started incorrectly using it and introducing things like CarbonApplicationContext/ContextHolder/HolderBase and so on.
Azeez On Fri, Sep 14, 2012 at 5:40 PM, Pradeep Fernando <[email protected]> wrote: > Hi all, > > agree with the versioning and p2 repo issues. But when it comes to the > actual effort that needs to be put in to carry out this refactoring : > *trivial*. > > You just have to replace the, > > SuperTenantCarbonContext class with PrivilegedCarbonContext > replace usage of CarbonContextHolder with CarbonContext. > > This is basically a find and replace across platform. This wont affect > the stability of the platform either, as we are not doing any drastic > changes to the usage. However Once the refactoring is done, > > the pros include : > > cleaner API - earlier it was only senaka who could decrypt this API. :) > performance enhancements due to minimal delegations. > This will also help our efforts of introducing pooling/optimizations > in this space. - targeting stratos > > > if the people are so concerned about versioning incompatibilities, we > could always go for a 4.1.0 > > thanks, > --Pradeep > _______________________________________________ > 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
