billierinaldi commented on a change in pull request #1450: Fix #1373 Support
the init scope in some VolumeChoosers
URL: https://github.com/apache/accumulo/pull/1450#discussion_r356146970
##########
File path:
server/base/src/main/java/org/apache/accumulo/server/ServiceEnvironmentImpl.java
##########
@@ -44,6 +44,11 @@ public ServiceEnvironmentImpl(ServerContext ctx) {
this.conf = new ConfigurationImpl(srvCtx.getConfiguration());
}
+ public ServiceEnvironmentImpl(ServerContext ctx, AccumuloConfiguration acfg)
{
+ this.srvCtx = ctx;
+ this.conf = new ConfigurationImpl(acfg);
+ }
+
Review comment:
Thanks for the new patch, @karthick-rn! I believe these lines can be removed
from ServiceEnvironmentImpl since they are no longer used.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
