[GitHub] [accumulo] ctubbsii commented on issue #886: Log if general.classpaths or general.dynamic.classpaths is set

[GitBox <gi...@apache.org>]

ctubbsii commented on issue #886: Log if general.classpaths or general.dynamic.classpaths is set
URL: https://github.com/apache/accumulo/issues/886#issuecomment-494158103
 
 
   @hkeebler The properties are deprecated as of #885. The preferred way of setting a class path is to set the CLASSPATH environment variable in `conf/accumulo-env.sh` (or whatever environment is appropriate for a user's particular deployment). This replaces the `general.classpaths` configuration property.
   
   As for the `general.dynamic.classpaths` property, that should also be deprecated. The dynamic class loading features are not, and have never been, stable in Accumulo. We were trying to provide a reasonable mechanism for users to drop in user code without restarting the servers, but this was always unreliable. Users can achieve a similar result by using a custom class loader (of their choosing) and configuring it appropriately, using built-in Java features to specify the system class loader with a system property: `-Djava.system.class.loader=com.test.YourCustomClassLoader`.
   
   So, for this issue, we just want to mark the properties deprecated, and then print a warning if they are set by the user, to highlight the fact that they are deprecated.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services