zentol commented on pull request #16191:
URL: https://github.com/apache/flink/pull/16191#issuecomment-865300941
> What happens if we use close in production code? Would everything fail
miserable with cryptic messages?
Production code has to ensure that the loaded plugin is no longer used
before closing the class loader. In practice I intend to call close when the
ClusterEntrypoint/TaskManagerRunner/MiniCluster is being shutdown, after the
RPCService has been shut down.
> Could we add @VisibleForTesting for now to avoid the issues?
`@VisibleForTesting` is not applicable in this case, because the MiniCluster
(the primary user) is _not_ test 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]
