Capitalize first letter in done logs for easier grepping (oracle#3769)

fixes oracle#3768
Akash3121 · Nov 19, 2021 · 1ac9223 · 1ac9223
1 parent 7ba25c1
commit 1ac9223
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/history/BoundaryChangesets.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/history/BoundaryChangesets.java
@@ -91,7 +91,7 @@ public synchronized List<String> getBoundaryChangesetIDs(String sinceRevision) t
         Collections.reverse(result);
 
         stat.report(LOGGER, Level.FINE,
-                String.format("done getting boundary changesets for ''%s'' (%d entries)",
+                String.format("Done getting boundary changesets for ''%s'' (%d entries)",
                         repository.getDirectoryName(), result.size()));
 
         return List.copyOf(result);

diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/history/HistoryGuru.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/history/HistoryGuru.java
@@ -854,7 +854,7 @@ public void invalidateRepositories(Collection<? extends RepositoryInfo> repos, C
         clear();
         newrepos.forEach((_key, repo) -> putRepository(repo));
 
-        elapsed.report(LOGGER, String.format("done invalidating %d repositories", newrepos.size()),
+        elapsed.report(LOGGER, String.format("Done invalidating %d repositories", newrepos.size()),
                 "history.repositories.invalidate");
     }
 

diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/index/Indexer.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/index/Indexer.java
@@ -945,7 +945,7 @@ public static void writeConfigToFile(RuntimeEnvironment env, String filename) th
         if (filename != null) {
             LOGGER.log(Level.INFO, "Writing configuration to {0}", filename);
             env.writeConfiguration(new File(filename));
-            LOGGER.info("Done...");
+            LOGGER.log(Level.INFO, "Done writing configuration to {0}", filename);
         }
     }
 
@@ -1042,14 +1042,14 @@ public void prepareIndexer(RuntimeEnvironment env,
         if (createHistoryCache) {
             // Even if history is disabled globally, it can be enabled for some repositories.
             if (repositories != null && !repositories.isEmpty()) {
-                LOGGER.log(Level.INFO, "Generating history cache for repositories: " +
+                LOGGER.log(Level.INFO, "Generating history cache for repositories: {0}",
                         String.join(",", repositories));
                 HistoryGuru.getInstance().createCache(repositories);
             } else {
                 LOGGER.log(Level.INFO, "Generating history cache for all repositories ...");
                 HistoryGuru.getInstance().createCache();
             }
-            LOGGER.info("Done...");
+            LOGGER.info("Done generating history cache");
         }
 
         if (createDict) {

diff --git a/tools/src/main/python/opengrok_tools/utils/command.py b/tools/src/main/python/opengrok_tools/utils/command.py
@@ -268,7 +268,7 @@ def close(self):
 
             self.logger.debug("Waiting for process with PID {}".format(p.pid))
             p.wait()
-            self.logger.debug("done waiting")
+            self.logger.debug("Done waiting")
 
             if self.timeout:
                 e = timeout_thread.get_exception()