You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "ivakegg (via GitHub)" <gi...@apache.org> on 2023/05/12 17:24:28 UTC
[GitHub] [accumulo] ivakegg opened a new issue, #3399: Access to AccumuloConfiguration when overridding classloader
ivakegg opened a new issue, #3399:
URL: https://github.com/apache/accumulo/issues/3399
In order to resolve our issue described in #3371, it was denoted that we override the class loader and hence the context class loader factory to be able to catch the errors being thrown that would halt the tservers. So I created such a mechanism and hit the problem where the ClassLoaderUtil does not pass the accumulo configuration to the factory if overridden. This is needed to be able to access the context classpaths currently configured. We have a good amount of infrastructure surrounding setting up this configuration as we add new contexts. Can we perhaps add a method to the ContextClassLoaderFactory to pass the accumulo configuration in, or at least the subset related to context configuration (general*.context.*).
Accumulo 2.1.1-SNAPSHOT
CentOS 7.3
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii commented on issue #3399: Access to AccumuloConfiguration when overridding classloader
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on issue #3399:
URL: https://github.com/apache/accumulo/issues/3399#issuecomment-1546503884
If you're using classpaths as the context, then the factory will already receive that context when it is asked for the context's corresponding context. However, I don't see a problem injecting a service environment.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ivakegg commented on issue #3399: Access to AccumuloConfiguration when overridding classloader
Posted by "ivakegg (via GitHub)" <gi...@apache.org>.
ivakegg commented on issue #3399:
URL: https://github.com/apache/accumulo/issues/3399#issuecomment-1546072071
Perhaps add a method on the ContextClassLoaderFactory to pass the accumulo configuration in, or perhaps a ServiceEnvironment implementation. I guess a ClassLoaderEnvironment mechanism would be the cleanest way to handle this.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] asfgit closed issue #3399: Access to AccumuloConfiguration when overridding classloader
Posted by "asfgit (via GitHub)" <gi...@apache.org>.
asfgit closed issue #3399: Access to AccumuloConfiguration when overridding classloader
URL: https://github.com/apache/accumulo/issues/3399
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org