[
https://issues.apache.org/jira/browse/CASSANDRA-2227?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13006643#comment-13006643
]
Matthew F. Dennis commented on CASSANDRA-2227:
----------------------------------------------
don't use thread.sleep() waiting for the future to be done, just call
future.get()
I wouldn't bother reading the cache directly from disk and checking the
contents, but rather force a save (the way you do) and then force C* to
completely load the cache as if it was opening the table for the first time and
then checking the size/contents via the actual cache in use. The patch
implements this at the end of the test anyway, so just assert there is nothing
in the cache, then create the CFS and verify the cache size/contents. Do
something similar for the row cache.
otherwise the patch looks good, but I would like to see these changes before
it's committed.
> add cache loading to row/key cache tests
> ----------------------------------------
>
> Key: CASSANDRA-2227
> URL: https://issues.apache.org/jira/browse/CASSANDRA-2227
> Project: Cassandra
> Issue Type: Test
> Components: Core
> Reporter: Jonathan Ellis
> Assignee: Pavel Yaskevich
> Priority: Minor
> Fix For: 0.7.5
>
> Attachments: CASSANDRA-2227.patch
>
> Original Estimate: 2h
> Remaining Estimate: 2h
>
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
