maedhroz commented on code in PR #4428:
URL: https://github.com/apache/cassandra/pull/4428#discussion_r2535880702
##########
src/java/org/apache/cassandra/db/ReadExecutionController.java:
##########
@@ -50,6 +61,13 @@ public class ReadExecutionController implements AutoCloseable
private final RepairedDataInfo repairedDataInfo;
private long oldestUnrepairedTombstone = Long.MAX_VALUE;
+ /*
+ * Track bulk transfers involved in the read, so we can do read
reconciliation.
+ * These come from the ViewFragment, not the SSTable read path, so bloom
filters + short-circuiting SSTable scans
+ * will still include the total set of relevant bulk transfers.
+ */
Review Comment:
nit: If you use a JavaDoc comment here, you can do fun stuff like `{@link
ColumnFamilyStore.ViewFragment}`...FWIW
--
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]
