[ 
https://issues.apache.org/jira/browse/CASSANDRA-15773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17101888#comment-17101888
 ] 

David Capwell commented on CASSANDRA-15773:
-------------------------------------------

bq. I've initialized the test with a fixed size to the pool

Sorry, must have missed that.

bq. I might not understand something but I thought that this would prevent such 
problems:

I most likely didn't look close enough, feel free to ignore me with regard to 
leaking.

Thanks for the new failure tests!

* 
https://github.com/apache/cassandra/compare/trunk...spmallette:CASSANDRA-15773-trunk#diff-790a43fcd6ed69f4122cd0fe205c847bR187-R198.
 You could use 
org.assertj.core.api.Assertions#assertThatThrownBy(org.assertj.core.api.ThrowableAssert.ThrowingCallable).

Other than that, this patch LGTM so I am +1.  Ill run it through CI to make 
sure its stable

> Add test to cover metrics related to the BufferPool
> ---------------------------------------------------
>
>                 Key: CASSANDRA-15773
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15773
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Test/unit
>            Reporter: Stephen Mallette
>            Assignee: Stephen Mallette
>            Priority: Normal
>
> At this time there do not appear to be unit tests to validate 
> {{BufferPoolMetrics}}.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to