[GitHub] mikewalch commented on a change in pull request #585: ServerContext is now a singleton class

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

mikewalch commented on a change in pull request #585: ServerContext is now a singleton class
URL: https://github.com/apache/accumulo/pull/585#discussion_r209278470
 
 

 ##########
 File path: server/base/src/main/java/org/apache/accumulo/server/constraints/MetadataConstraints.java
 ##########
 @@ -297,7 +296,7 @@ private static boolean isValidColumn(ColumnUpdate cu) {
   }
 
   protected Arbitrator getArbitrator() {
-    return new ZooArbitrator();
+    return new ZooArbitrator(ServerContext.getInstance());
 
 Review comment:
   I am not going to close #582 until all calls to getInstance() outside of main methods are removed and calls to getInstance() are converted to constructors.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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