turboFei commented on code in PR #3063:
URL: https://github.com/apache/incubator-kyuubi/pull/3063#discussion_r920642602
##########
kyuubi-server/src/main/scala/org/apache/kyuubi/engine/JpsApplicationOperation.scala:
##########
@@ -65,6 +65,9 @@ class JpsApplicationOperation extends ApplicationOperation {
val (id, _) = idAndCmd.splitAt(idAndCmd.indexOf(" "))
try {
s"kill -15 $id".lineStream
+ if (getEngine(tag).nonEmpty) {
+ killApplicationByTag(tag)
Review Comment:
seems the first process is transient and just for spark-submit arguments
parser.
So, if we try to kill,
we might get the first process id, but it is transient, so if we kill the
process id, it might say that: Kill - No such process.
But the real process is still running, because it has another pid.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
