Increase polling frequency to ensure that Java processes are detected (microsoft#416)

testforstephen · web-flow · commit f8f34858446e · 2022-05-31T15:17:57.000+08:00
* Increase polling frequency to ensure that Java processes are detected
diff --git a/com.microsoft.java.debug.core/src/main/java/com/microsoft/java/debug/core/adapter/handler/LaunchUtils.java b/com.microsoft.java.debug.core/src/main/java/com/microsoft/java/debug/core/adapter/handler/LaunchUtils.java
@@ -117,7 +117,7 @@ public static ProcessHandle findJavaProcessInTerminalShell(long shellPid, String
         ProcessHandle shellProcess = ProcessHandle.of(shellPid).orElse(null);
         if (shellProcess != null) {
             int retry = 0;
-            final int INTERVAL = 100;
+            final int INTERVAL = 20/*ms*/;
             final int maxRetries = timeout / INTERVAL;
             final boolean isCygwinShell = isCygwinShell(shellProcess.info().command().orElse(null));
             while (retry <= maxRetries) {
@@ -127,11 +127,17 @@ public static ProcessHandle findJavaProcessInTerminalShell(long shellPid, String
                 }).findFirst();
 
                 if (subProcessHandle.isPresent()) {
+                    if (retry > 0) {
+                        logger.info("Retried " + retry + " times to find Java subProcess.");
+                    }
                     logger.info("shellPid: " + shellPid + ", javaPid: " + subProcessHandle.get().pid());
                     return subProcessHandle.get();
                 } else if (isCygwinShell) {
                     long javaPid = findJavaProcessByCygwinPsCommand(shellProcess, javaCommand);
                     if (javaPid > 0) {
+                        if (retry > 0) {
+                            logger.info("Retried " + retry + " times to find Java subProcess.");
+                        }
                         logger.info("[Cygwin Shell] shellPid: " + shellPid + ", javaPid: " + javaPid);
                         return ProcessHandle.of(javaPid).orElse(null);
                     }
@@ -147,8 +153,9 @@ public static ProcessHandle findJavaProcessInTerminalShell(long shellPid, String
                 } catch (InterruptedException e) {
                     // do nothing
                 }
-                logger.info("Retry to find Java subProcess of shell pid " + shellPid);
             }
+
+            logger.info("Retried " + retry + " times but failed to find Java subProcess of shell pid " + shellPid);
         }
 
         return null;
