Remove noise & extra log statement.

Signed-off-by: Marc Handalian <[email protected]>
opensearch-project · Sep 7, 2022 · d8f2cab · d8f2cab
1 parent 2c276c4
commit d8f2cab
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/server/src/main/java/org/opensearch/index/engine/NRTReplicationEngine.java b/server/src/main/java/org/opensearch/index/engine/NRTReplicationEngine.java
@@ -350,7 +350,6 @@ public SafeCommitInfo getSafeCommitInfo() {
 
     @Override
     protected final void closeNoLock(String reason, CountDownLatch closedLatch) {
-        logger.info("Closing");
         if (isClosed.compareAndSet(false, true)) {
             assert rwl.isWriteLockedByCurrentThread() || failEngineLock.isHeldByCurrentThread()
                 : "Either the write lock must be held or the engine must be currently be failing itself";

diff --git a/server/src/test/java/org/opensearch/index/store/StoreTests.java b/server/src/test/java/org/opensearch/index/store/StoreTests.java
@@ -658,16 +658,16 @@ public void testRecoveryDiff() throws IOException, InterruptedException {
         Store.RecoveryDiff newCommitDiff = newCommitMetadata.recoveryDiff(metadata);
         if (delFile != null) {
             assertThat(newCommitDiff.identical.size(), equalTo(newCommitMetadata.size() - 5)); // segments_N, del file, cfs, cfe, si for the
-            // new segment
+                                                                                               // new segment
             assertThat(newCommitDiff.different.size(), equalTo(1)); // the del file must be different
             assertThat(newCommitDiff.different.get(0).name(), endsWith(".liv"));
             assertThat(newCommitDiff.missing.size(), equalTo(4)); // segments_N,cfs, cfe, si for the new segment
         } else {
             assertThat(newCommitDiff.identical.size(), equalTo(newCommitMetadata.size() - 4)); // segments_N, cfs, cfe, si for the new
-            // segment
+                                                                                               // segment
             assertThat(newCommitDiff.different.size(), equalTo(0));
             assertThat(newCommitDiff.missing.size(), equalTo(4)); // an entire segment must be missing (single doc segment got dropped) plus
-            // the commit is different
+                                                                  // the commit is different
         }
 
         deleteContent(store.directory());