(WIP) Try to reproduce and fix race condition (#1647) #1656
Conversation
|
|
||
| Awaitility.await() | ||
| .atMost(30, TimeUnit.SECONDS) | ||
| .untilAsserted(() -> assertThat(responses.size()).isEqualTo(2 * ROUNDS)); |
There was a problem hiding this comment.
Fails here only getting 1000 -- but I don't think that actually reproduces the NPE we want to catch; both since no matching error found in logs and since suggested fix for QueryProcessor access does not solve the fail.
So I suspect assertion here is incorrect.
There was a problem hiding this comment.
(and as per my later comment, NPE not shown because PR did NOT UPDATE Cassandra dep which comes from different PR).
|
@tatu-at-datastax I don't know what was the idea, but here you don't have upgrade to C* |
|
@ivansenic Correct, the original PR (#1647) has that. Ohh... shoot. I am an idiot. Wasted lots of time due to not upgrading version of 4.0. |
|
Let s close this one, you can port anything you want to the original PR. |
Yup. |
What this PR does:
Which issue(s) this PR fixes:
Fixes #1647
Checklist