dddyszy commented on a change in pull request #4936:
URL: 
https://github.com/apache/incubator-dolphinscheduler/pull/4936#discussion_r590214019



##########
File path: 
dolphinscheduler-server/src/main/java/org/apache/dolphinscheduler/server/utils/ProcessUtils.java
##########
@@ -59,7 +59,7 @@
     /**
      * Expression of PID recognition in Windows scene
      */
-    private static final Pattern WINDOWSATTERN = Pattern.compile("(\\d+)");
+    private static final Pattern WINDOWSATTERN = 
Pattern.compile("\\((\\d+)\\)");

Review comment:
       matcher.group returns the match string in (), website regex utils find 
the whole match string,they are not the same.You can try it in java code




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to