yifan-c commented on PR #93: URL: https://github.com/apache/cassandra-analytics/pull/93#issuecomment-3097726389
@liucao-dd, thanks for paying attention to the details! Just to clarify that in the vnode setup, the `List<Instance> instances` for `calculateTokenRanges` is a list of vnodes sorted by their token, not the physical C* nodes/processes. For example, a cluster of 12 C* nodes with vnode=4 should produce the list of 48 `Instance`. With the clarification, do you think the calculation is still wrong? The calculation here is to find the total owned range (primary + replicated) of each vnode. -- 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: commits-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
