tbonelee commented on PR #5310: URL: https://github.com/apache/zeppelin/pull/5310#issuecomment-4995076161
Thanks for the fix! This same change (moving `docker.close()` into a `finally` block in `stop()`) already landed in `main` via #5302. See the current `DockerInterpreterProcess.stop()`. That PR also handles the `start()` orphaned-container case, so this looks like a duplicate. Mind closing it? Thanks for spotting this! -- 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]
