maedhroz commented on code in PR #2492:
URL: https://github.com/apache/cassandra/pull/2492#discussion_r1264245433
##########
src/java/org/apache/cassandra/index/SecondaryIndexManager.java:
##########
@@ -1706,6 +1754,7 @@ public void handleNotification(INotification
notification, Object sender)
indexes.values()
.stream()
.filter(Index::shouldBuildBlocking)
+ .filter(i ->
!i.isSSTableAttached())
Review Comment:
If we're here, it's either after streaming or via `SSTableImporter`. SAI
will already have written its indexes in the streaming case (entire file
streaming, or writing incrementally via `SSTableFlushObserver`), and this patch
redirects the importer case.
So, this is here to just make sure we ignore SAI indexes.
--
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]
