[ https://issues.apache.org/jira/browse/CASSANDRA-18521?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Caleb Rackliffe updated CASSANDRA-18521: ---------------------------------------- Status: Ready to Commit (was: Review In Progress) Yup, go for it! > Unify CQLTester#waitForIndex and SAITester#waitForIndexQueryable > ---------------------------------------------------------------- > > Key: CASSANDRA-18521 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18521 > Project: Cassandra > Issue Type: Improvement > Components: Feature/SAI > Reporter: Andres de la Peña > Assignee: Andres de la Peña > Priority: Normal > Fix For: NA > > > It has been > [mentioned|https://github.com/maedhroz/cassandra/pull/11#discussion_r1190166765] > in CASSANDRA-18217 that {{CQLTester#waitForIndex}} and > {{SAITester#waitForIndexQueryable}} can be unified into a single method. > The same discussion mentions that it's easy to forget to call those methods > after calling {{CQLTester#createIndex}}. So that method can be renamed to > {{createIndexAsync}}, and we can create a new method that creates the index > and waits for it ({{createIndexSync}}, {{createIndexAndWaitForQueryable}}, or > something like that). -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org