yifan-c commented on a change in pull request #856:
URL: https://github.com/apache/cassandra/pull/856#discussion_r574118526
##########
File path: src/java/org/apache/cassandra/db/ColumnFamilyStore.java
##########
@@ -2806,4 +2806,25 @@ public boolean getNeverPurgeTombstones()
{
return neverPurgeTombstones;
}
-}
\ No newline at end of file
+
+ /**
+ * Check SSTables whether or not they are misplaced.
+ * @return true if any of the SSTables is misplaced.
+ * If all SSTables are correctly placed or the partitioner does
not support splitting, it returns false.
+ */
+ @Override
+ public boolean isSSTableMisplaced()
Review comment:
I'd prefer to have the method name start with a verb.
How about `hasMisplacedSSTables()`? It has the same meaning as
`isAnySSTableMisplaced()`, but shorter.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
