-
Notifications
You must be signed in to change notification settings - Fork 909
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support Curator X discovery and others #2673
Labels
Milestone
Comments
minwoox
added a commit
to minwoox/armeria
that referenced
this issue
May 29, 2020
Motivation: Related line#2673. It will be nice if we support Curator-X-Discovery and Spring Cloud Zookeeper. Modifications: - Add `InstanceSpec` and `DiscoverySpec` to specify whether use default format or Curator compatible format. - (Breaking) - `NodeValueCodec` is gone. - You now have to use `InstanceSpec` and `DiscoverySpec` to encode and decode. - `ZooKeeperEndpointGroupBuilder.codec(...)` and `ZooKeeperUpatingListenerBuilder.codec(...)` are gone as well. Result: - You can now use Armeria client and server with Curator-X-Discovery.
trustin
pushed a commit
that referenced
this issue
Jun 3, 2020
Related: #2673 Motivation: It will be nice if we support Curator Service Discovery and Spring Cloud Zookeeper. Modifications: - Add `ZookeeperRegistrationSpec` and `ZookeeperDiscoverySpec` to specify whether use legacy format or Curator compatible format. - (Breaking) You should specify `ZookeeperRegistrationSpec` When creating `ZookeeperUpdatingListener`. - (Breaking) You should specify `ZookeeperDiscoverySpec` When creating `ZookeeperEndpointGroup`. - (Breaking) - `NodeValueCodec` is gone. - You now have to use `ZookeeperRegistrationSpec` and `ZookeeperDiscoverySpec` to encode and decode. - `ZooKeeperEndpointGroupBuilder.codec(...)` and `ZooKeeperUpatingListenerBuilder.codec(...)` are gone as well. Result: - You can now use Armeria client and server with Curator Service Discovery.
minwoox
added a commit
to minwoox/armeria
that referenced
this issue
Jun 9, 2020
Motivation: It will be nice if we support `ServerSets` service discovery. Modifications: - Add `ZooKeeperDiscoverySpec` and `ZooKeeperRegistrationSpec` for `ServerSets` - Rename `...Zookeeper...` to `...ZooKeeper...` (K is capital letter.) Result: - Close line#2673 - You can now register an Armeria server and discover services with the `ServerSets` compatible format.
minwoox
added a commit
to minwoox/armeria
that referenced
this issue
Jun 9, 2020
Motivation: It will be nice if we support `ServerSets` service discovery. Modifications: - Add `ZooKeeperDiscoverySpec` and `ZooKeeperRegistrationSpec` for `ServerSets` - Rename `...Zookeeper...` to `...ZooKeeper...` (K is capital letter.) Result: - Close line#2673 - You can now register an Armeria server and discover services with the `ServerSets` compatible format.
trustin
pushed a commit
that referenced
this issue
Jun 16, 2020
Motivation: It will be nice if we support `ServerSets` service discovery. Modifications: - Add `ZooKeeperDiscoverySpec` and `ZooKeeperRegistrationSpec` for `ServerSets` - Rename `...Zookeeper...` to `...ZooKeeper...` (K is capital letter.) Result: - Close #2673 - You can now register an Armeria server and discover services with the `ServerSets` compatible format.
fmguerreiro
pushed a commit
to fmguerreiro/armeria
that referenced
this issue
Sep 19, 2020
Related: line#2673 Motivation: It will be nice if we support Curator Service Discovery and Spring Cloud Zookeeper. Modifications: - Add `ZookeeperRegistrationSpec` and `ZookeeperDiscoverySpec` to specify whether use legacy format or Curator compatible format. - (Breaking) You should specify `ZookeeperRegistrationSpec` When creating `ZookeeperUpdatingListener`. - (Breaking) You should specify `ZookeeperDiscoverySpec` When creating `ZookeeperEndpointGroup`. - (Breaking) - `NodeValueCodec` is gone. - You now have to use `ZookeeperRegistrationSpec` and `ZookeeperDiscoverySpec` to encode and decode. - `ZooKeeperEndpointGroupBuilder.codec(...)` and `ZooKeeperUpatingListenerBuilder.codec(...)` are gone as well. Result: - You can now use Armeria client and server with Curator Service Discovery.
fmguerreiro
pushed a commit
to fmguerreiro/armeria
that referenced
this issue
Sep 19, 2020
Motivation: It will be nice if we support `ServerSets` service discovery. Modifications: - Add `ZooKeeperDiscoverySpec` and `ZooKeeperRegistrationSpec` for `ServerSets` - Rename `...Zookeeper...` to `...ZooKeeper...` (K is capital letter.) Result: - Close line#2673 - You can now register an Armeria server and discover services with the `ServerSets` compatible format.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, We support service discovery with Zookeeper using our own format, which means that it's not compatible with others such as:
In order to be the framework well suited everywhere, We should generalize and provide a way to use whatever a user wants.
The text was updated successfully, but these errors were encountered: