maedhroz commented on code in PR #2492:
URL: https://github.com/apache/cassandra/pull/2492#discussion_r1267381474
##########
src/java/org/apache/cassandra/index/Index.java:
##########
@@ -341,13 +341,21 @@ default public Callable<?> getPreJoinTask(boolean
hadBootstrap)
* Return true if this index can be built or rebuilt when the index
manager determines it is necessary. Returning
* false enables the index implementation (or some other component) to
control if and when SSTable data is
* incorporated into the index.
- *
+ * <p>
* This is called by SecondaryIndexManager in buildIndexBlocking,
buildAllIndexesBlocking and rebuildIndexesBlocking
* where a return value of false causes the index to be exluded from the
set of those which will process the
* SSTable data.
* @return if the index should be included in the set which processes
SSTable data, false otherwise.
*/
- public boolean shouldBuildBlocking();
+ boolean shouldBuildBlocking();
+
+ /**
+ * @return true if the index builds SSTable-attached on-disk components
+ */
+ default boolean isSSTableAttached()
Review Comment:
Yeah, I think we're all hesitant to make many more changes around this or
the index API when we know those two are basically defunct.
--
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]
