maedhroz commented on a change in pull request #1334:
URL: https://github.com/apache/cassandra/pull/1334#discussion_r753448433
##########
File path:
test/distributed/org/apache/cassandra/distributed/test/SecondaryIndexTest.java
##########
@@ -43,9 +45,10 @@
private static final int NUM_NODES = 3;
private static final int REPLICATION_FACTOR = 1;
private static final String CREATE_TABLE = "CREATE TABLE %s(k int, v int,
PRIMARY KEY (k))";
- private static final String CREATE_INDEX = "CREATE INDEX v_index ON %s(v)";
+ private static final String CREATE_INDEX = "CREATE INDEX v_index_%d ON
%s(v)";
Review comment:
Note: It didn't matter for the correctness of the test, but it seemed
good hygiene to suffix the index names, which I think are global to the
keyspace.
--
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]
