From b092bf28788c76b6ca1cf8285592d08cc832e8e4 Mon Sep 17 00:00:00 2001
From: Henning Baldersheim <balder@yahoo-inc.com>
Date: Thu, 11 Jan 2024 12:18:14 +0100
Subject: Avoid deprecated methods

---
 .../com/yahoo/system/execution/ProcessExecutor.java   | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

(limited to 'vespajlib/src/main/java/com')

diff --git a/vespajlib/src/main/java/com/yahoo/system/execution/ProcessExecutor.java b/vespajlib/src/main/java/com/yahoo/system/execution/ProcessExecutor.java
index 691a5844596..fe3bca07845 100644
--- a/vespajlib/src/main/java/com/yahoo/system/execution/ProcessExecutor.java
+++ b/vespajlib/src/main/java/com/yahoo/system/execution/ProcessExecutor.java
@@ -12,6 +12,7 @@ import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.InputStream;
 import java.nio.charset.StandardCharsets;
+import java.time.Duration;
 import java.util.Optional;
 import java.util.concurrent.TimeUnit;
 
@@ -24,11 +25,11 @@ import java.util.concurrent.TimeUnit;
 public class ProcessExecutor {
 
     public static class Builder {
-        private final int timeoutSeconds;
+        private final Duration timeout;
         private int[] successExitCodes;
 
-        public Builder(int timeoutSeconds) {
-            this.timeoutSeconds = timeoutSeconds;
+        public Builder(Duration timeout) {
+            this.timeout = timeout;
         }
 
         public Builder successExitCodes(int... successExitCodes) {
@@ -37,16 +38,16 @@ public class ProcessExecutor {
         }
 
         public ProcessExecutor build() {
-            return new ProcessExecutor(timeoutSeconds, successExitCodes);
+            return new ProcessExecutor(timeout, successExitCodes);
         }
     }
 
-    private ProcessExecutor(int timeoutSeconds, int[] successExitCodes) {
-        this.timeoutSeconds = timeoutSeconds;
+    private ProcessExecutor(Duration timeout, int[] successExitCodes) {
+        this.timeout = timeout;
         this.successExitCodes = successExitCodes;
     }
 
-    public final int timeoutSeconds;
+    private final Duration timeout;
     private final int[] successExitCodes;
 
     /**
@@ -68,9 +69,9 @@ public class ProcessExecutor {
         ByteArrayOutputStream processErr = new ByteArrayOutputStream();
         ByteArrayOutputStream processOut = new ByteArrayOutputStream();
 
-        DefaultExecutor executor = new DefaultExecutor();
+        DefaultExecutor executor = DefaultExecutor.builder().get();
         executor.setStreamHandler(createStreamHandler(processOut, processErr, processInput));
-        ExecuteWatchdog watchDog = new ExecuteWatchdog(TimeUnit.SECONDS.toMillis(timeoutSeconds));
+        ExecuteWatchdog watchDog = ExecuteWatchdog.builder().setTimeout(timeout).get();
         executor.setWatchdog(watchDog);
         executor.setExitValues(successExitCodes);
 
-- 
cgit v1.2.3