LadyForest commented on pull request #15087: URL: https://github.com/apache/flink/pull/15087#issuecomment-791190531
Hi, @fsk119 thanks for your suggestion. You're right that we don't need to fetch result from `executor#listModules.` I think we could remove it from the `Exectuor` as interface method, but keep it in `LocalExecutor` and mark with `@VisiableForTesting`, since this will make tests on `LOAD/UNLOAD MODULE` and `USE MODULES` much easier. What do you think? ---------------------------------------------------------------- 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]
