timothy65535 edited a comment on issue #918: URL: https://github.com/apache/incubator-kyuubi/issues/918#issuecomment-897323135
Thanks share your idea. Conceptually, ServiceDiscovery contains `register` and `discovery` two modules. In the codebase, most of the designs of`ServiceDiscovery`, `KyuubiServiceDiscovery`, `EngineServiceDiscovery` are ok, maybe just a little change. So, have an idea that we can minimize the issue scope and do these separately. We can start with the first and second points, they will not affect the following future change. -- 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]
