wangyang0918 commented on pull request #15501: URL: https://github.com/apache/flink/pull/15501#issuecomment-816434710
I am also not suggesting to implement our own shared watch. It just feels like we are doing some duplicated things that shared informer already supports. If we plan to use the shared informer to replace current individual watch, then I believe we do not need to keep the original interfaces(e.g. `FlinkKubeClient#watchConfigMaps`). After then we could make the implementation cleaner and easier to test(e.g. a ITCase for the behavior of shared informer). -- 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]
