maedhroz commented on code in PR #3575:
URL: https://github.com/apache/cassandra/pull/3575#discussion_r1800280359
##########
src/java/org/apache/cassandra/db/SinglePartitionReadCommand.java:
##########
@@ -1266,6 +1272,32 @@ public boolean isRangeRequest()
return false;
}
+ /*
+ * The execution method does not need to perform reconciliation so the
read command
+ * should execute in a mannager suited to not needing reconciliation. Such
as when
+ * executing transactionally at a single replica and doing an index scan
where the index
+ * scan should not return extra rows and expect post filtering at the
coordinator.
+ */
+ public SinglePartitionReadCommand withoutReconciliation()
+ {
+ if (indexQueryPlan() == null)
+ return this;
Review Comment:
Yeah, if you had a partition-restricted non-index filtering query, the row
filter would be non-empty and you'd want to get past this short-circuit. Looks
like you've already updated this in your last commit though.
--
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]
