HSEARCH-4069 Avoid to init timing source if not needed

hibernate · Nov 5, 2020 · d8b66fb · d8b66fb
1 parent ab48be8
commit d8b66fb
Showing 1 changed file with 14 additions and 5 deletions.
diff --git a/engine/src/main/java/org/hibernate/search/engine/search/timeout/spi/TimeoutManager.java b/engine/src/main/java/org/hibernate/search/engine/search/timeout/spi/TimeoutManager.java
@@ -73,20 +73,25 @@ public TimeoutManager(TimingSource timingSource, Long timeoutValue, TimeUnit tim
 		this.type = type;
 		this.deadline = timeoutMs == null ? null : new DynamicDeadline();
 
-		timingSource.ensureTimeEstimateIsInitialized();
+		if ( requireMonotonicTimeEstimate() ) {
+			timingSource.ensureTimeEstimateIsInitialized();
+		}
 	}
 
 	/**
 	 * we start counting from this method call (if needed)
 	 */
 	public void start() {
-		this.monotonicTimeEstimateStart = timingSource.monotonicTimeEstimate();
-		this.nanoTimeStart = timingSource.nanoTime();
+		if ( requireMonotonicTimeEstimate() ) {
+			monotonicTimeEstimateStart = timingSource.monotonicTimeEstimate();
+		}
+
+		nanoTimeStart = timingSource.nanoTime();
 	}
 
 	public void stop() {
-		this.monotonicTimeEstimateStart = null;
-		this.nanoTimeStart = null;
+		monotonicTimeEstimateStart = null;
+		nanoTimeStart = null;
 	}
 
 	public TimingSource timingSource() {
@@ -155,6 +160,10 @@ protected long elapsedTimeEstimateMillis() {
 		return timingSource.monotonicTimeEstimate() - monotonicTimeEstimateStart;
 	}
 
+	private boolean requireMonotonicTimeEstimate() {
+		return !Type.NONE.equals( type );
+	}
+
 	final class DynamicDeadline implements Deadline {
 		boolean timedOut = false;