maedhroz commented on code in PR #3046:
URL: https://github.com/apache/cassandra/pull/3046#discussion_r1454224835


##########
test/unit/org/apache/cassandra/io/sstable/CQLSSTableWriterTest.java:
##########
@@ -63,38 +77,29 @@
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
 
-public class CQLSSTableWriterTest
+@Ignore
+public abstract class CQLSSTableWriterTest
 {
     private static final AtomicInteger idGen = new AtomicInteger(0);
-    private String keyspace;
-    private String table;
-    private String qualifiedTable;
-    private File dataDir;
+    private static final int NUMBER_WRITES_IN_RUNNABLE = 10;
 
-    static
-    {
-        DatabaseDescriptor.daemonInitialization();
-    }
+    private Supplier<CQLSSTableWriter.Builder> defaultBuilderSupplier = () -> 
CQLSSTableWriter.builder().removeSchemaAfterDone(false);
 
     @Rule
     public TemporaryFolder tempFolder = new TemporaryFolder();
 
-    @BeforeClass
-    public static void setup() throws Exception
-    {
-        CommitLog.instance.start();
-        SchemaLoader.cleanupAndLeaveDirs();
-        Keyspace.setInitialized();
-        StorageService.instance.initServer();
-    }
+    private String keyspace;
+    protected String table;
+    private String qualifiedTable;
+    protected File dataDir;
 
     @Before
     public void perTestSetup() throws IOException

Review Comment:
   ```suggestion
       public void perTestSetup()
   ```



-- 
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]

Reply via email to