fsk119 commented on pull request #15087: URL: https://github.com/apache/flink/pull/15087#issuecomment-791182171
Hi, @LadyForest. Thanks for your contribution. For the commit 96fe72d, we leverage `TableEnvironment` to list the modules. It seems the `Executor` doesn't need `listModules` method anymore. I think it's time to delete the method now. 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]
