-Re: Review Request 17987: Followup patch incorporting Jay's suggestions
This is an automatically generated e-mail. To reply, visit:
This is good, can you break it into another test. Something like testRoundRobinIsFair()?
Also can you remove the repeated code. Should be something like
val part = partitioner.partition(...)
assertEquals((n + i) % 2, part)
- Jay Kreps
On Feb. 12, 2014, 3:14 a.m., Guozhang Wang wrote: