maedhroz commented on code in PR #2329:
URL: https://github.com/apache/cassandra/pull/2329#discussion_r1192830494
##########
test/unit/org/apache/cassandra/index/sai/cql/BaseDataModel.java:
##########
@@ -176,7 +176,7 @@ public void createIndexes(Executor tester) throws Throwable
if (!skipColumns.contains(column.left))
{
executeLocalIndexed(tester, String.format(template,
column.left, indexedTable, column.left));
- tester.waitForIndexQueryable(KEYSPACE, indexedTable);
+ tester.waitForTableIndexesQueryable(KEYSPACE, indexedTable);
Review Comment:
I think the original intention of this and the call on 304 was to wait for
the single column index just created. In that case, we need to either use
`waitForIndexQueryable(keyspace, indexName)` or move the
`waitForTableIndexesQueryable()` outside the loop. The former might be better
isolated. WDYT?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
