mirror of https://github.com/apache/cassandra
CASSANDRA-19911 Remove MemtableSizeTest
This is a ninja / leftover from CASSANDRA-17298, we replaced this test but forgot to remove it as was done in other branches.
This commit is contained in:
Stefan Miklosovic
parent
6ec8c5bb81
commit
ddb01d2f8e
|
|
@ -1,156 +0,0 @@
|
||||||
/*
|
|
||||||
* Licensed to the Apache Software Foundation (ASF) under one
|
|
||||||
* or more contributor license agreements. See the NOTICE file
|
|
||||||
* distributed with this work for additional information
|
|
||||||
* regarding copyright ownership. The ASF licenses this file
|
|
||||||
* to you under the Apache License, Version 2.0 (the
|
|
||||||
* "License"); you may not use this file except in compliance
|
|
||||||
* with the License. You may obtain a copy of the License at
|
|
||||||
*
|
|
||||||
* http://www.apache.org/licenses/LICENSE-2.0
|
|
||||||
*
|
|
||||||
* Unless required by applicable law or agreed to in writing, software
|
|
||||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
|
||||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
||||||
* See the License for the specific language governing permissions and
|
|
||||||
* limitations under the License.
|
|
||||||
*/
|
|
||||||
|
|
||||||
package org.apache.cassandra.cql3;
|
|
||||||
|
|
||||||
import java.util.List;
|
|
||||||
|
|
||||||
import com.google.common.base.Throwables;
|
|
||||||
import com.google.common.collect.ImmutableList;
|
|
||||||
import org.junit.Assert;
|
|
||||||
import org.junit.BeforeClass;
|
|
||||||
import org.junit.Test;
|
|
||||||
import org.junit.runner.RunWith;
|
|
||||||
import org.junit.runners.Parameterized;
|
|
||||||
|
|
||||||
import org.slf4j.Logger;
|
|
||||||
import org.slf4j.LoggerFactory;
|
|
||||||
|
|
||||||
import org.apache.cassandra.Util;
|
|
||||||
import org.apache.cassandra.config.DatabaseDescriptor;
|
|
||||||
import org.apache.cassandra.db.ColumnFamilyStore;
|
|
||||||
import org.apache.cassandra.db.Keyspace;
|
|
||||||
import org.apache.cassandra.db.memtable.Memtable;
|
|
||||||
import org.apache.cassandra.utils.FBUtilities;
|
|
||||||
import org.apache.cassandra.utils.ObjectSizes;
|
|
||||||
|
|
||||||
@RunWith(Parameterized.class)
|
|
||||||
public class MemtableSizeTest extends CQLTester
|
|
||||||
{
|
|
||||||
static final Logger logger = LoggerFactory.getLogger(MemtableSizeTest.class);
|
|
||||||
|
|
||||||
static final int partitions = 50_000;
|
|
||||||
static final int rowsPerPartition = 4;
|
|
||||||
|
|
||||||
static final int deletedPartitions = 10_000;
|
|
||||||
static final int deletedRows = 5_000;
|
|
||||||
|
|
||||||
@Parameterized.Parameter(0)
|
|
||||||
public String memtableClass;
|
|
||||||
|
|
||||||
@Parameterized.Parameter(1)
|
|
||||||
public int differencePerPartition;
|
|
||||||
|
|
||||||
@Parameterized.Parameters(name = "{0}")
|
|
||||||
public static List<Object[]> parameters()
|
|
||||||
{
|
|
||||||
return ImmutableList.of(new Object[]{"skiplist", 50},
|
|
||||||
new Object[]{"skiplist_sharded", 60});
|
|
||||||
}
|
|
||||||
|
|
||||||
// must be within 50 bytes per partition of the actual size
|
|
||||||
final long MAX_DIFFERENCE_PARTITIONS = (partitions + deletedPartitions + deletedRows);
|
|
||||||
|
|
||||||
@BeforeClass
|
|
||||||
public static void setUp()
|
|
||||||
{
|
|
||||||
CQLTester.setUpClass();
|
|
||||||
CQLTester.prepareServer();
|
|
||||||
CQLTester.disablePreparedReuseForTest();
|
|
||||||
logger.info("setupClass done.");
|
|
||||||
}
|
|
||||||
|
|
||||||
@Test
|
|
||||||
public void testSize()
|
|
||||||
{
|
|
||||||
Util.flakyTest(this::testSizeFlaky, 2, "Fails occasionally, see CASSANDRA-16684");
|
|
||||||
}
|
|
||||||
|
|
||||||
private void testSizeFlaky()
|
|
||||||
{
|
|
||||||
try
|
|
||||||
{
|
|
||||||
String keyspace = createKeyspace("CREATE KEYSPACE %s with replication = { 'class' : 'SimpleStrategy', 'replication_factor' : 1 } and durable_writes = false");
|
|
||||||
String table = createTable(keyspace, "CREATE TABLE %s ( userid bigint, picid bigint, commentid bigint, PRIMARY KEY(userid, picid))" +
|
|
||||||
" with compression = {'enabled': false}" +
|
|
||||||
" and memtable = '" + memtableClass + "'");
|
|
||||||
execute("use " + keyspace + ';');
|
|
||||||
|
|
||||||
String writeStatement = "INSERT INTO " + table + "(userid,picid,commentid)VALUES(?,?,?)";
|
|
||||||
|
|
||||||
ColumnFamilyStore cfs = Keyspace.open(keyspace).getColumnFamilyStore(table);
|
|
||||||
cfs.disableAutoCompaction();
|
|
||||||
Util.flush(cfs);
|
|
||||||
|
|
||||||
long deepSizeBefore = ObjectSizes.measureDeep(cfs.getTracker().getView().getCurrentMemtable());
|
|
||||||
logger.info("Memtable deep size before {}\n",
|
|
||||||
FBUtilities.prettyPrintMemory(deepSizeBefore));
|
|
||||||
long i;
|
|
||||||
long limit = partitions;
|
|
||||||
logger.info("Writing {} partitions of {} rows", partitions, rowsPerPartition);
|
|
||||||
for (i = 0; i < limit; ++i)
|
|
||||||
{
|
|
||||||
for (long j = 0; j < rowsPerPartition; ++j)
|
|
||||||
execute(writeStatement, i, j, i + j);
|
|
||||||
}
|
|
||||||
|
|
||||||
logger.info("Deleting {} partitions", deletedPartitions);
|
|
||||||
limit += deletedPartitions;
|
|
||||||
for (; i < limit; ++i)
|
|
||||||
{
|
|
||||||
// no partition exists, but we will create a tombstone
|
|
||||||
execute("DELETE FROM " + table + " WHERE userid = ?", i);
|
|
||||||
}
|
|
||||||
|
|
||||||
logger.info("Deleting {} rows", deletedRows);
|
|
||||||
limit += deletedRows;
|
|
||||||
for (; i < limit; ++i)
|
|
||||||
{
|
|
||||||
// no row exists, but we will create a tombstone (and partition)
|
|
||||||
execute("DELETE FROM " + table + " WHERE userid = ? AND picid = ?", i, 0L);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!cfs.getLiveSSTables().isEmpty())
|
|
||||||
logger.info("Warning: " + cfs.getLiveSSTables().size() + " sstables created.");
|
|
||||||
|
|
||||||
Memtable memtable = cfs.getTracker().getView().getCurrentMemtable();
|
|
||||||
Memtable.MemoryUsage usage = Memtable.getMemoryUsage(memtable);
|
|
||||||
long actualHeap = usage.ownsOnHeap;
|
|
||||||
logger.info("Memtable in {} mode: {} ops, {} serialized bytes, {}\n",
|
|
||||||
DatabaseDescriptor.getMemtableAllocationType(),
|
|
||||||
memtable.operationCount(),
|
|
||||||
FBUtilities.prettyPrintMemory(memtable.getLiveDataSize()),
|
|
||||||
usage);
|
|
||||||
|
|
||||||
long deepSizeAfter = ObjectSizes.measureDeep(memtable);
|
|
||||||
logger.info("Memtable deep size {}\n",
|
|
||||||
FBUtilities.prettyPrintMemory(deepSizeAfter));
|
|
||||||
|
|
||||||
long expectedHeap = deepSizeAfter - deepSizeBefore;
|
|
||||||
String message = String.format("Expected heap usage close to %s, got %s.\n",
|
|
||||||
FBUtilities.prettyPrintMemory(expectedHeap),
|
|
||||||
FBUtilities.prettyPrintMemory(actualHeap));
|
|
||||||
logger.info(message);
|
|
||||||
Assert.assertTrue(message, Math.abs(actualHeap - expectedHeap) <= MAX_DIFFERENCE_PARTITIONS * differencePerPartition);
|
|
||||||
}
|
|
||||||
catch (Throwable throwable)
|
|
||||||
{
|
|
||||||
Throwables.propagate(throwable);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
Loading…
Reference in New Issue