maedhroz commented on code in PR #4055:
URL: https://github.com/apache/cassandra/pull/4055#discussion_r2033625597


##########
src/java/org/apache/cassandra/index/sai/plan/StorageAttachedIndexSearcher.java:
##########
@@ -291,7 +291,7 @@ private void fillNextSelectedKeysInPartition(DecoratedKey 
partitionKey, List<Pri
                 if (key == null)
                     break;
 
-                if (queryController.doesNotSelect(key) || key.equals(lastKey))
+                if (queryController.doesNotSelect(key) || key.equals(lastKey, 
false))

Review Comment:
   The idea with these two changes is that, if we've already processed a STATIC 
key in a partition, there is no need to process any normal rows in the same 
partition (because we'll have post-filtered the whole partition already).



-- 
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]

Reply via email to