Skip to content

Commit cc8c6f3

Browse files
author
Vladimir Kotal
committed
compute rebuild duration correctly
fixes #3337
1 parent 1255d4b commit cc8c6f3

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

suggester/src/main/java/org/opengrok/suggest/Suggester.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -186,9 +186,7 @@ public void init(final Collection<NamedIndexDir> luceneIndexes) {
186186
submitInitIfIndexExists(executor, indexDir);
187187
}
188188

189-
Duration duration = Duration.between(start, Instant.now());
190-
suggesterInitTimer.record(duration);
191-
shutdownAndAwaitTermination(executor, duration, "Suggester successfully initialized");
189+
shutdownAndAwaitTermination(executor, start, "Suggester successfully initialized");
192190
initDone.countDown();
193191
}
194192
}
@@ -252,10 +250,13 @@ private boolean indexExists(final Path indexDir) throws IOException {
252250
}
253251
}
254252

255-
private void shutdownAndAwaitTermination(final ExecutorService executorService, Duration duration, final String logMessageOnSuccess) {
253+
private void shutdownAndAwaitTermination(final ExecutorService executorService, Instant start,
254+
final String logMessageOnSuccess) {
256255
executorService.shutdown();
257256
try {
258257
executorService.awaitTermination(awaitTerminationTime.toMillis(), TimeUnit.MILLISECONDS);
258+
Duration duration = Duration.between(start, Instant.now());
259+
suggesterInitTimer.record(duration);
259260
logger.log(Level.INFO, "{0} (took {1})", new Object[]{logMessageOnSuccess,
260261
DurationFormatUtils.formatDurationWords(duration.toMillis(),
261262
true, true)});
@@ -295,9 +296,8 @@ public void rebuild(final Collection<NamedIndexDir> indexDirs) {
295296
}
296297
}
297298

298-
Duration duration = Duration.between(start, Instant.now());
299-
suggesterRebuildTimer.record(duration);
300-
shutdownAndAwaitTermination(executor, duration, "Suggesters for " + indexDirs + " were successfully rebuilt");
299+
shutdownAndAwaitTermination(executor, start,
300+
"Suggesters for " + indexDirs + " were successfully rebuilt");
301301
}
302302

303303
rebuildLock.lock();

0 commit comments

Comments
 (0)