Copilot commented on code in PR #7971:
URL: https://github.com/apache/geode/pull/7971#discussion_r2612374932
##########
geode-logging/src/main/java/org/apache/geode/logging/internal/OSProcess.java:
##########
@@ -135,8 +135,6 @@ private static native int bgexecInternal(String[] cmdarray,
String workdir, Stri
* inherited from the parent process and will overwrite same keys
* @return the process id of the created process; -1 on failure
* @exception SecurityException if the current thread cannot create a
subprocess.
Review Comment:
The @exception SecurityException documentation should be removed since the
SecurityManager check code has been removed. This method no longer throws
SecurityException (it's not declared in the throws clause either). The
SecurityException tag is outdated and misleading.
```suggestion
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
