stonewhitener opened a new pull request, #70:
URL: https://github.com/apache/cassandra-accord/pull/70
* Fixed a problem that `writeCount` might be `0` even though `isWrite` is
true.
* Since the number of nodes is determined by `[rf, 3 * rf)`,
`Math.max(nodes.size() + 1, rf)` always returns `nodes.size() + 1` when
determining the number of ranges in a cluster. Therefore, the unnecessary `max`
has been removed.
* Furthermore, I would suggest the number of ranges should include
`nodes.size()` therefore the following line:
```java
IntHashKey.ranges(random.nextInt(nodes.size() + 1, nodes.size() * 3))
```
should be:
```java
IntHashKey.ranges(random.nextInt(nodes.size(), nodes.size() * 3))
```
--
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]
