dcapwell commented on code in PR #4081: URL: https://github.com/apache/cassandra/pull/4081#discussion_r2044981199
########## test/distributed/org/apache/cassandra/distributed/test/sai/SAIUtil.java: ########## @@ -86,23 +86,26 @@ public static void assertIndexQueryable(Cluster cluster, String keyspace, String */ private static void assertIndexesQueryable(Cluster cluster, String keyspace, final Iterable<String> indexes) { - IInvokableInstance localNode = cluster.get(1); Review Comment: this is the reason for the flakeyness. SAIUtil checks the status on node1, which is based off gossip; so we know 2/3 have completed the index... but we don't know that 2/3 know about each other's completed status! So you query node2 and it says node3 is in index rebuild, so can't process the request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org