Changes log levels and improves log statements

Signed-off-by: Darshit Chanpura <[email protected]>
opensearch-project · Dec 20, 2024 · 193e846 · 193e846
1 parent f514859
commit 193e846
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/src/main/java/org/opensearch/security/OpenSearchSecurityPlugin.java b/src/main/java/org/opensearch/security/OpenSearchSecurityPlugin.java
@@ -2128,7 +2128,7 @@ public void onNodeStarted(DiscoveryNode localNode) {
             String resourceIndex = resourcePlugin.getResourceIndex();
 
             this.indicesToListen.add(resourceIndex);
-            log.info("Preparing to listen to index: {} of plugin: {}", resourceIndex, resourcePlugin);
+            log.warn("Security plugin started listening to index: {} of plugin: {}", resourceIndex, resourcePlugin);
         }
 
         final Set<ModuleInfo> securityModules = ReflectionHelper.getModulesLoaded();
@@ -2148,7 +2148,6 @@ public Collection<Class<? extends LifecycleComponent>> getGuiceServiceClasses()
 
         final List<Class<? extends LifecycleComponent>> services = new ArrayList<>(1);
         services.add(GuiceHolder.class);
-        log.info("Guice service classes loaded");
         return services;
     }
 

diff --git a/src/main/java/org/opensearch/security/resources/ResourceSharingIndexListener.java b/src/main/java/org/opensearch/security/resources/ResourceSharingIndexListener.java
@@ -42,9 +42,7 @@ public class ResourceSharingIndexListener implements IndexingOperationListener {
     private ResourceSharingIndexListener() {}
 
     public static ResourceSharingIndexListener getInstance() {
-
         return ResourceSharingIndexListener.INSTANCE;
-
     }
 
     /**
@@ -122,11 +120,9 @@ public void postDelete(ShardId shardId, Engine.Delete delete, Engine.DeleteResul
 
         boolean success = this.resourceSharingIndexHandler.deleteResourceSharingRecord(resourceId, resourceIndex);
         if (success) {
-            log.info("Successfully deleted resource sharing entries for resource {}", resourceId);
+            log.info("Successfully deleted resource sharing entry for resource {}", resourceId);
         } else {
             log.info("Failed to delete resource sharing entry for resource {}", resourceId);
         }
-
     }
-
 }