Skip to content

Commit

Permalink
Fix ReadCommandTest
Browse files Browse the repository at this point in the history
patch by Kurt Greaves; reviewed by jasobrown for CASSANDRA-14234
  • Loading branch information
kurt authored and jasobrown committed Feb 14, 2018
1 parent d10e6ac commit 1d506f9
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 3 deletions.
1 change: 1 addition & 0 deletions CHANGES.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
3.11.2
* Fix ReadCommandTest (CASSANDRA-14234)
* Remove trailing period from latency reports at keyspace level (CASSANDRA-14233)
* Backport CASSANDRA-13080: Use new token allocation for non bootstrap case as well (CASSANDRA-14212)
* Remove dependencies on JVM internal classes from JMXServerUtils (CASSANDRA-14173)
Expand Down
31 changes: 28 additions & 3 deletions test/unit/org/apache/cassandra/db/ReadCommandTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,8 @@ public class ReadCommandTest
private static final String CF1 = "Standard1";
private static final String CF2 = "Standard2";
private static final String CF3 = "Standard3";
private static final String CF4 = "Standard4";
private static final String CF5 = "Standard5";

@BeforeClass
public static void defineSchema() throws ConfigurationException
Expand All @@ -85,12 +87,35 @@ public static void defineSchema() throws ConfigurationException
.addRegularColumn("e", AsciiType.instance)
.addRegularColumn("f", AsciiType.instance).build();

CFMetaData metadata4 = CFMetaData.Builder.create(KEYSPACE, CF4)
.addPartitionKey("key", BytesType.instance)
.addClusteringColumn("col", AsciiType.instance)
.addRegularColumn("a", AsciiType.instance)
.addRegularColumn("b", AsciiType.instance)
.addRegularColumn("c", AsciiType.instance)
.addRegularColumn("d", AsciiType.instance)
.addRegularColumn("e", AsciiType.instance)
.addRegularColumn("f", AsciiType.instance).build();

CFMetaData metadata5 = CFMetaData.Builder.create(KEYSPACE, CF5)
.addPartitionKey("key", BytesType.instance)
.addClusteringColumn("col", AsciiType.instance)
.addRegularColumn("a", AsciiType.instance)
.addRegularColumn("b", AsciiType.instance)
.addRegularColumn("c", AsciiType.instance)
.addRegularColumn("d", AsciiType.instance)
.addRegularColumn("e", AsciiType.instance)
.addRegularColumn("f", AsciiType.instance).build();


SchemaLoader.prepareServer();
SchemaLoader.createKeyspace(KEYSPACE,
KeyspaceParams.simple(1),
metadata1,
metadata2,
metadata3);
metadata3,
metadata4,
metadata5);
}

@Test
Expand Down Expand Up @@ -260,7 +285,7 @@ public void close()
@Test
public void testCountDeletedRows() throws Exception
{
ColumnFamilyStore cfs = Keyspace.open(KEYSPACE).getColumnFamilyStore(CF3);
ColumnFamilyStore cfs = Keyspace.open(KEYSPACE).getColumnFamilyStore(CF4);

String[][][] groups = new String[][][] {
new String[][] {
Expand Down Expand Up @@ -308,7 +333,7 @@ public void testCountDeletedRows() throws Exception
@Test
public void testCountWithNoDeletedRow() throws Exception
{
ColumnFamilyStore cfs = Keyspace.open(KEYSPACE).getColumnFamilyStore(CF3);
ColumnFamilyStore cfs = Keyspace.open(KEYSPACE).getColumnFamilyStore(CF5);

String[][][] groups = new String[][][] {
new String[][] {
Expand Down

0 comments on commit 1d506f9

Please sign in to comment.