search

scylladb / scylla-jmx

Scylla JMX proxy
GNU Affero General Public License v3.0
28 stars 52 forks source link

StorageService.java: Use the endpoint for getRangeToEndpointMap #143

Closed amnonh closed 3 years ago

amnonh commented 3 years ago

After implementing range_to_endpoint_map endpoint update the API call to it.

Fixes #36

Signed-off-by: Amnon Heiman amnon@scylladb.com

penberg commented 3 years ago

@amnonh Is the API side implemented in scylla.git already? Which commit(s)?

amnonh commented 3 years ago

The unimplemented API is there (i.e. you will get an unimplemented exception just like the situation today).

The actual implementation is in the PR https://github.com/scylladb/scylla/pull/7329

penberg commented 3 years ago

@amnonh Thanks. I guess I grepped for the wrong thing in scylla.git because I thought range_to_endpoint_map did not exist at all...