-
Notifications
You must be signed in to change notification settings - Fork 504
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
Serverset namer can't resolve shard fragment #2390
Comments
If this functionality is useful I'd be happy to code it up and submit a PR. |
@JustinVenus a contribution would be awesome! I'm not familiar with shards, is this for zookeeper? |
@cpretzer serversets have a json payload on the znode in zookeeper. The format is roughly like the following. {"status": "ALIVE", "additionalEndpoints": {"http": {"host": "host1", "port": 31238}}, "serviceEndpoint": {"host": "host1", "port": 31238}, "shard": 0} Most serverset implementations (that I am familiar with) are able to advertise the shard (aka instanceId). The default/upstream finagle implementation of the ServersetNamer allows one to craft a dtab to specify a given shard/instanceId. My use case is to prefer a primary instanceId with the ability to failover to a standby like the following example.
|
Got it! Thanks for teaching me something new 😀 |
Resolved with #2391 |
Issue Type:
What happened:
I was not able to resolve a name in the form of
/service/prod/read:http#0
What you expected to happen:
I expected to get shard member
0
for service labelhttp
of thePath(/service/prod/read)
.How to reproduce it (as minimally and precisely as possible):
dtab playground on the admin interface
with a shard like#0
Anything else we need to know?:
It seems like the functionality could be added here and would allow the namer to behave inline with upstream finagle.
Environment:
The text was updated successfully, but these errors were encountered: