Github user dineshjoshi commented on a diff in the pull request:
https://github.com/apache/cassandra/pull/279#discussion_r225404002
--- Diff:
test/burn/org/apache/cassandra/utils/memory/LongBufferPoolTest.java ---
@@ -129,16 +165,22 @@ void testOne() throws Exception
return;
}
- BufferPool.put(buffer);
- burn.add(buffer);
+ // 50/50 chance of returning the buffer from the
producer thread, or
+ // pass it on to the consumer.
+ if (rand.nextBoolean()) {
--- End diff --
According to the code style, you can skip braces here. We also follow
follow Sun Style so braces go on new lines.
Reference:
http://cassandra.apache.org/doc/latest/development/code_style.html
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
