Remove verbose log (#5959)

* Fix #5955 - Remove verbose log
hapifhir · May 24, 2024 · 93e1019 · 93e1019
1 parent 0fe3380
commit 93e1019
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/...s/src/main/resources/ca/uhn/hapi/fhir/changelog/7_4_0/5959-remove-overly-verbose-log.yaml b/...s/src/main/resources/ca/uhn/hapi/fhir/changelog/7_4_0/5959-remove-overly-verbose-log.yaml
@@ -0,0 +1,4 @@
+---
+type: fix
+issue: 5959
+title: "An overly verbose log in the JPA server about deleting stale searches has been reduced to DEBUG level."
diff --git a/...jpaserver-base/src/main/java/ca/uhn/fhir/jpa/search/cache/DatabaseSearchCacheSvcImpl.java b/...jpaserver-base/src/main/java/ca/uhn/fhir/jpa/search/cache/DatabaseSearchCacheSvcImpl.java
@@ -220,7 +220,6 @@ private void commitOpenChanges() {
  // flush to force Hibernate to actually get a connection from the pool
  myEntityManager.flush();
  // get our connection from the underlying Hibernate session, and commit
- //noinspection resource
  myEntityManager.unwrap(Session.class).doWork(Connection::commit);
  }
 
@@ -252,17 +251,19 @@ private int deleteMarkedSearchesInBatches() {
  flushSearchAndIncludeDeletes();
 
  int deletedCount = deletedCounter.get();
-
- ourLog.info("Deleted {} expired searches", deletedCount);
+ if (deletedCount > 0) {
+ ourLog.debug("Deleted {} expired searches", deletedCount);
+ }
 
  return deletedCount;
  }
 
  /**
  * Schedule theSearchPid for deletion assuming it has theNumberOfResults SearchResults attached.
- *
+ * <p>
  * We accumulate a batch of search pids for deletion, and then do a bulk DML as we reach a threshold number
  * of SearchResults.
+ * </p>
  *
  * @param theSearchPid pk of the Search
  * @param theNumberOfResults the number of SearchResults attached