ACCUMULO-3728 Add javadoc to CompactionStrategyConfig Also removes unnecessary "empty" lines in/around javadoc.
Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/2b0ed2f6 Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/2b0ed2f6 Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/2b0ed2f6 Branch: refs/heads/master Commit: 2b0ed2f6a7adf6d77381c494c1eb682dc457c024 Parents: d71dbe6 Author: Josh Elser <els...@apache.org> Authored: Tue Apr 14 17:34:45 2015 -0400 Committer: Josh Elser <els...@apache.org> Committed: Wed Apr 15 20:36:38 2015 -0400 ---------------------------------------------------------------------- .../accumulo/core/client/admin/CompactionStrategyConfig.java | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/accumulo/blob/2b0ed2f6/core/src/main/java/org/apache/accumulo/core/client/admin/CompactionStrategyConfig.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/accumulo/core/client/admin/CompactionStrategyConfig.java b/core/src/main/java/org/apache/accumulo/core/client/admin/CompactionStrategyConfig.java index 75aa107..c04c175 100644 --- a/core/src/main/java/org/apache/accumulo/core/client/admin/CompactionStrategyConfig.java +++ b/core/src/main/java/org/apache/accumulo/core/client/admin/CompactionStrategyConfig.java @@ -24,20 +24,19 @@ import java.util.Map; import com.google.common.base.Preconditions; /** + * Configuration object which describes how a Compaction is run. Configuration objects are dependent upon the CompactionStrategy running insider the server. + * This class is used in conjunction with {@link CompactionConfig}. * * @since 1.7.0 */ - public class CompactionStrategyConfig { private String className; private Map<String,String> options = Collections.emptyMap(); /** - * * @param className * The name of a class that implements org.apache.accumulo.tserver.compaction.CompactionStrategy. This class must be exist on tservers. */ - public CompactionStrategyConfig(String className) { Preconditions.checkNotNull(className); this.className = className; @@ -56,7 +55,6 @@ public class CompactionStrategyConfig { * The default is an empty map. * @return this */ - public CompactionStrategyConfig setOptions(Map<String,String> opts) { Preconditions.checkNotNull(opts); this.options = new HashMap<>(opts); @@ -64,10 +62,8 @@ public class CompactionStrategyConfig { } /** - * * @return The previously set options. Returns an unmodifiable map. The default is an empty map. */ - public Map<String,String> getOptions() { return Collections.unmodifiableMap(options); }
