skoppu22 commented on PR #222: URL: https://github.com/apache/cassandra-analytics/pull/222#issuecomment-5620903097
> Hi @skoppu22 Follow-up is up: [apache/cassandra-sidecar#380 (CASSSIDECAR-495)](https://github.com/apache/cassandra-sidecar/pull/380). > > It adds `sidecarInstanceId` to both the ring and token-range-replicas responses, reported per node/replica by the Sidecar itself (via its local instances config), instead of the client trying to derive it from a hostname it observes. > > This covers the shared-address case you flagged here: multiple Cassandra instances fronted by one Sidecar address (e.g. behind a load balancer). The client no longer needs a hostname-keyed lookup at all, it reads the correct `instanceId` straight off the ring/ token-range response for each node Sorry for the delay, I am taking a look now -- 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]
