use default empty implementation

idegtiarenko · Oct 26, 2023 · 6bc84bd · 6bc84bd
1 parent e0002b2
commit 6bc84bd
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 11 deletions.
diff --git a/...c/internalClusterTest/java/org/elasticsearch/action/admin/cluster/node/tasks/TasksIT.java b/...c/internalClusterTest/java/org/elasticsearch/action/admin/cluster/node/tasks/TasksIT.java
@@ -447,9 +447,6 @@ public void onTaskRegistered(Task task) {
                             }
                         }
                     }
-
-                    @Override
-                    public void onTaskUnregistered(Task task) {}
                 });
             }
             // Need to run the task in a separate thread because node client's .execute() is blocked by our task listener
@@ -601,9 +598,6 @@ private <T> void waitForCompletionTestCase(boolean storeResult, Function<TaskId,
             CountDownLatch waitForWaitingToStart = new CountDownLatch(1);
             for (TransportService transportService : internalCluster().getInstances(TransportService.class)) {
                 ((MockTaskManager) transportService.getTaskManager()).addListener(new MockTaskManagerListener() {
-                    @Override
-                    public void onTaskRegistered(Task task) {}
-
                     @Override
                     public void onTaskUnregistered(Task task) {
                         waitForWaitingToStart.countDown();

diff --git a/server/src/test/java/org/elasticsearch/cluster/service/MasterServiceTests.java b/server/src/test/java/org/elasticsearch/cluster/service/MasterServiceTests.java
@@ -228,9 +228,6 @@ public void testCreatesChildTaskForPublishingClusterState() throws Exception {
             public void onTaskRegistered(Task task) {
                 registeredActions.add(task.getAction());
             }
-
-            @Override
-            public void onTaskUnregistered(Task task) {}
         });
 
         final CountDownLatch latch = new CountDownLatch(1);

diff --git a/test/framework/src/main/java/org/elasticsearch/test/tasks/MockTaskManagerListener.java b/test/framework/src/main/java/org/elasticsearch/test/tasks/MockTaskManagerListener.java
@@ -15,9 +15,9 @@
  * Listener for task registration/unregistration
  */
 public interface MockTaskManagerListener {
-    void onTaskRegistered(Task task);
+    default void onTaskRegistered(Task task) {};
 
-    void onTaskUnregistered(Task task);
+    default void onTaskUnregistered(Task task) {};
 
     default void subscribeForRemovedTasks(RemovedTaskListener removedTaskListener) {}
 }