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


##########
src/java/org/apache/cassandra/service/reads/DataResolver.java:
##########
@@ -275,6 +275,10 @@ private PartitionIterator 
resolveWithReplicaFilteringProtection(E replicas, Repa
 
     private  UnaryOperator<PartitionIterator> 
preCountFilterForReplicaFilteringProtection()
     {
+        // Key columns are immutable and should never need to participate in 
replica filtering
+        if (!command.rowFilter().hasNonKeyExpression())
+            return results -> results;

Review Comment:
   For some reason I had never back-ported this to 5.0



-- 
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: pr-unsubscr...@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org

Reply via email to