bereng commented on code in PR #1891:
URL: https://github.com/apache/cassandra/pull/1891#discussion_r1108173461
##########
test/unit/org/apache/cassandra/io/sstable/LegacySSTableTest.java:
##########
@@ -598,10 +598,12 @@ private static void loadLegacyTable(String tablePattern,
String legacyVersion) t
* <p>
* Run this test alone (e.g. from your IDE) when a new version is
introduced or format changed
* during development. I.e. remove the {@code @Ignore} annotation
temporarily.
+ *
+ * *************RUN THIS TEST ALONE to avoid unwanted compactions,
flushes, etc to interfere
Review Comment:
I have torn thoughts here: being in the same class sort of makes sense so
you don't forget about it. But it's too easy to mess things around as this
_has_ to be ran alone :shrug: I hope my comment is good enough to avoid the
trouble I ran into to the next developer here.
--
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]
