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

Reply via email to