From aebb7c602fa15425e175afe5aba0ac20e8bc73be Mon Sep 17 00:00:00 2001 From: Gaurav Bafna Date: Tue, 18 Jun 2024 14:51:02 +0530 Subject: [PATCH] Address PR comments Signed-off-by: Gaurav Bafna --- .../main/java/org/opensearch/repositories/Repository.java | 4 +++- .../opensearch/repositories/RepositoriesServiceTests.java | 5 ----- .../org/opensearch/index/shard/RestoreOnlyRepository.java | 5 ----- 3 files changed, 3 insertions(+), 11 deletions(-) diff --git a/server/src/main/java/org/opensearch/repositories/Repository.java b/server/src/main/java/org/opensearch/repositories/Repository.java index dd8480c9705c0..ed30aad7b4dd2 100644 --- a/server/src/main/java/org/opensearch/repositories/Repository.java +++ b/server/src/main/java/org/opensearch/repositories/Repository.java @@ -214,7 +214,9 @@ default void deleteSnapshotsAndReleaseLockFiles( /** * Returns low priority upload throttle time in nanoseconds */ - long getLowPriorityRemoteUploadThrottleTimeInNanos(); + default long getLowPriorityRemoteUploadThrottleTimeInNanos() { + return 0; + } /** * Returns restore throttle time in nanoseconds diff --git a/server/src/test/java/org/opensearch/repositories/RepositoriesServiceTests.java b/server/src/test/java/org/opensearch/repositories/RepositoriesServiceTests.java index ed180c90f3c78..43ebb86fd5342 100644 --- a/server/src/test/java/org/opensearch/repositories/RepositoriesServiceTests.java +++ b/server/src/test/java/org/opensearch/repositories/RepositoriesServiceTests.java @@ -699,11 +699,6 @@ public long getRemoteUploadThrottleTimeInNanos() { return 0; } - @Override - public long getLowPriorityRemoteUploadThrottleTimeInNanos() { - return 0; - } - @Override public long getRemoteDownloadThrottleTimeInNanos() { return 0; diff --git a/test/framework/src/main/java/org/opensearch/index/shard/RestoreOnlyRepository.java b/test/framework/src/main/java/org/opensearch/index/shard/RestoreOnlyRepository.java index b01dc8a7b7072..be2f895301396 100644 --- a/test/framework/src/main/java/org/opensearch/index/shard/RestoreOnlyRepository.java +++ b/test/framework/src/main/java/org/opensearch/index/shard/RestoreOnlyRepository.java @@ -155,11 +155,6 @@ public long getRemoteUploadThrottleTimeInNanos() { return 0; } - @Override - public long getLowPriorityRemoteUploadThrottleTimeInNanos() { - return 0; - } - @Override public long getRemoteDownloadThrottleTimeInNanos() { return 0;