You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ct...@apache.org on 2014/08/23 00:06:37 UTC

[8/9] git commit: ACCUMULO-3019 "Un-deprecate" ServerConfiguration

ACCUMULO-3019 "Un-deprecate" ServerConfiguration

  Suppress all the unnecessary warnings noise due to the deprecation of this
  class without removing existing usage. One cannot reasonably consider it
  deprecated if we are still using it so extensively. This only applies to the
  current branch. It is deprecated in future branches. This commit should be
  merged to newer branches with the "ours" merge strategy.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/705a60d1
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/705a60d1
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/705a60d1

Branch: refs/heads/master
Commit: 705a60d19e55c3499c168d3dd3ad5215dd8487c4
Parents: 45d61cb
Author: Christopher Tubbs <ct...@apache.org>
Authored: Fri Aug 22 18:01:33 2014 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Fri Aug 22 18:01:33 2014 -0400

----------------------------------------------------------------------
 .../org/apache/accumulo/server/conf/ServerConfiguration.java    | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/705a60d1/server/base/src/main/java/org/apache/accumulo/server/conf/ServerConfiguration.java
----------------------------------------------------------------------
diff --git a/server/base/src/main/java/org/apache/accumulo/server/conf/ServerConfiguration.java b/server/base/src/main/java/org/apache/accumulo/server/conf/ServerConfiguration.java
index 0793177..50dec57 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/conf/ServerConfiguration.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/conf/ServerConfiguration.java
@@ -25,9 +25,9 @@ import org.apache.accumulo.core.conf.SiteConfiguration;
 import org.apache.accumulo.core.data.KeyExtent;
 
 /**
- * @deprecated Use {@link ServerConfigurationFactory} instead.
+ * Prefer {@link ServerConfigurationFactory} over this class, due to this class being deprecated in future versions with the addition of ACCUMULO-2615. It is
+ * left un-deprecated here, due to the fact that doing so would generate too much churn and noise. See ACCUMULO-3019.
  */
-@Deprecated
 public class ServerConfiguration {
 
   private static SecurityPermission CONFIGURATION_PERMISSION = new SecurityPermission("configurationPermission");
@@ -72,6 +72,7 @@ public class ServerConfiguration {
   static void expireAllTableObservers() {
     ServerConfigurationFactory.expireAllTableObservers();
   }
+
   private final ServerConfigurationFactory scf;
 
   public ServerConfiguration(Instance instance) {