This is an automated email from the ASF dual-hosted git repository.

dlmarion pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
     new c14a97fe8c Deprecate rate limiting properties (#3882)
c14a97fe8c is described below

commit c14a97fe8ca91bd6bc5bba3a2f2ba1094f11fac6
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Tue Oct 24 10:15:36 2023 -0400

    Deprecate rate limiting properties (#3882)
---
 core/src/main/java/org/apache/accumulo/core/conf/Property.java         | 3 +++
 .../apache/accumulo/core/util/compaction/CompactionServicesConfig.java | 1 +
 .../org/apache/accumulo/test/compaction/CompactionRateLimitingIT.java  | 1 +
 3 files changed, 5 insertions(+)

diff --git a/core/src/main/java/org/apache/accumulo/core/conf/Property.java 
b/core/src/main/java/org/apache/accumulo/core/conf/Property.java
index a4b627068c..903c5d7443 100644
--- a/core/src/main/java/org/apache/accumulo/core/conf/Property.java
+++ b/core/src/main/java/org/apache/accumulo/core/conf/Property.java
@@ -573,6 +573,7 @@ public enum Property {
   
TSERV_COMPACTION_SERVICE_ROOT_PLANNER("tserver.compaction.major.service.root.planner",
       DefaultCompactionPlanner.class.getName(), PropertyType.CLASSNAME,
       "Compaction planner for root tablet service", "2.1.0"),
+  @Deprecated(since = "3.1", forRemoval = true)
   
TSERV_COMPACTION_SERVICE_ROOT_RATE_LIMIT("tserver.compaction.major.service.root.rate.limit",
 "0B",
       PropertyType.BYTES,
       "Maximum number of bytes to read or write per second over all major"
@@ -591,6 +592,7 @@ public enum Property {
   
TSERV_COMPACTION_SERVICE_META_PLANNER("tserver.compaction.major.service.meta.planner",
       DefaultCompactionPlanner.class.getName(), PropertyType.CLASSNAME,
       "Compaction planner for metadata table", "2.1.0"),
+  @Deprecated(since = "3.1", forRemoval = true)
   
TSERV_COMPACTION_SERVICE_META_RATE_LIMIT("tserver.compaction.major.service.meta.rate.limit",
 "0B",
       PropertyType.BYTES,
       "Maximum number of bytes to read or write per second over all major"
@@ -609,6 +611,7 @@ public enum Property {
   
TSERV_COMPACTION_SERVICE_DEFAULT_PLANNER("tserver.compaction.major.service.default.planner",
       DefaultCompactionPlanner.class.getName(), PropertyType.CLASSNAME,
       "Planner for default compaction service.", "2.1.0"),
+  @Deprecated(since = "3.1", forRemoval = true)
   
TSERV_COMPACTION_SERVICE_DEFAULT_RATE_LIMIT("tserver.compaction.major.service.default.rate.limit",
       "0B", PropertyType.BYTES,
       "Maximum number of bytes to read or write per second over all major"
diff --git 
a/core/src/main/java/org/apache/accumulo/core/util/compaction/CompactionServicesConfig.java
 
b/core/src/main/java/org/apache/accumulo/core/util/compaction/CompactionServicesConfig.java
index c6e1cea55d..b3f911c467 100644
--- 
a/core/src/main/java/org/apache/accumulo/core/util/compaction/CompactionServicesConfig.java
+++ 
b/core/src/main/java/org/apache/accumulo/core/util/compaction/CompactionServicesConfig.java
@@ -44,6 +44,7 @@ public class CompactionServicesConfig {
 
   public static final CompactionServiceId DEFAULT_SERVICE = 
CompactionServiceId.of("default");
 
+  @SuppressWarnings("removal")
   private long getDefaultThroughput() {
     return ConfigurationTypeHelper
         
.getMemoryAsBytes(Property.TSERV_COMPACTION_SERVICE_DEFAULT_RATE_LIMIT.getDefaultValue());
diff --git 
a/test/src/main/java/org/apache/accumulo/test/compaction/CompactionRateLimitingIT.java
 
b/test/src/main/java/org/apache/accumulo/test/compaction/CompactionRateLimitingIT.java
index 0b4280f8d1..46e1060e24 100644
--- 
a/test/src/main/java/org/apache/accumulo/test/compaction/CompactionRateLimitingIT.java
+++ 
b/test/src/main/java/org/apache/accumulo/test/compaction/CompactionRateLimitingIT.java
@@ -39,6 +39,7 @@ public class CompactionRateLimitingIT extends 
ConfigurableMacBase {
   public static final long BYTES_TO_WRITE = 10 * 1024 * 1024;
   public static final long RATE = 1 * 1024 * 1024;
 
+  @SuppressWarnings("removal")
   protected Property getThroughputProp() {
     return Property.TSERV_COMPACTION_SERVICE_DEFAULT_RATE_LIMIT;
   }

Reply via email to