ruanwenjun commented on PR #1030: URL: https://github.com/apache/incubator-eventmesh/pull/1030#issuecomment-1189722366
> > I don't think it's time to do this change, we didn't register EventMesh runtime server in ZK. > > I have previously submitted a PR support ZK registry #974 . My thinking is that the type of registry currently supported is not critical, it may be important to design the nameResolver and loadbalance of client well, subsequent registries can be expanded on this basis. This is just my personal opinion, can you give me some suggestions ? You need to submit a mail to dev to discuss this, since this will make a great impact to EM. I agree we can add this feature, but you still need to send proposal(This is important), and we need to put the logic in your ZK registry, the client should only rely on registry to get/notify the change of server address. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
