Raul,

I agree with you. I remember a discussion like this on the dev list a while
ago, but it looks like it led to nowhere. I created a ticket:
https://issues.apache.org/jira/browse/IGNITE-1930, let's list the desired
scenarios of user class-loading and required changes there.

--AG

2015-11-13 20:39 GMT+03:00 Raul Kripalani <[email protected]>:

> Hi team,
>
> I'm seeing instances of ClassLoaders being kept as instance variables in
> many places. It looks a bit ad-hoc.
>
> What surprises me is that the user-set ClassLoader
> (IgniteConfiguration#setClassLoader) is ignored in many places where, in my
> opinion, it should be preferred, e.g.:
>
> * Optimized Marshaller
> * U.gridClassLoader() and its many usages.
> * etc.
>
> Shouldn't Ignite use the user-preferred classloader for everything, and
> only fallback to getClass().getClassLoader() and other methods as a
> default?
>
> Currently the user's ClassLoader is only used from these spots:
>
> * GridDeploymentManager
> * GridDeploymentPerVersionStore
> * CacheObjectImpl
> * GridCacheDeploymentManager
>
> *Raúl Kripalani*
> PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and
> Messaging Engineer
> http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani
> http://blog.raulkr.net | twitter: @raulvk
>

Reply via email to