adelapena commented on code in PR #1891:
URL: https://github.com/apache/cassandra/pull/1891#discussion_r1110067703
##########
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:
Not a big deal, but I think it's clearer to have the comment next to the
`@Ignore` tag, since it explains that tag. The `TODO` prefix is highlighted and
shown in code analyzers by some IDEs, which might be useful to remind us that
at some point we have to fix the test (separate ticket, etc.).
--
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]
