maedhroz commented on code in PR #3584:
URL: https://github.com/apache/cassandra/pull/3584#discussion_r1779440436
##########
src/java/org/apache/cassandra/service/reads/AbstractReadExecutor.java:
##########
@@ -431,7 +431,8 @@ public void awaitResponses(boolean
logBlockingReadRepairAttempt) throws ReadTime
// return immediately, or begin a read repair
if (digestResolver.responsesMatch())
{
- setResult(digestResolver.getData());
+ PartitionIterator data = digestResolver.getData();
+
setResult(command.filterForReplicaFilteringProtection().apply(data));
Review Comment:
I think we ultimately only want to use a single partition read command if we
have a strict `RowFilter`. It's at least an incremental improvement that
doesn't require us to worry too much about the short reads potentially caused
by non-strict replica filtering coupled w/ strict coordinator filtering.
--
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]
