You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@eventmesh.apache.org by GitBox <gi...@apache.org> on 2022/07/27 14:05:27 UTC
[GitHub] [incubator-eventmesh] kirinzhu commented on pull request #1030: [ISSUE #1016] init loadBalance for grpc producer
kirinzhu commented on PR #1030:
URL: https://github.com/apache/incubator-eventmesh/pull/1030#issuecomment-1196807186
> > > > 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.
>
> Hi, I have sent the email some time ago. I wonder what is happening now. 
Sorry for not sending you a read receipt. We are discussing about it, and will respond you as soon as possible.
--
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: dev-unsubscribe@eventmesh.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: dev-help@eventmesh.apache.org