dcapwell commented on code in PR #2190:
URL: https://github.com/apache/cassandra/pull/2190#discussion_r1128363975
##########
test/unit/org/apache/cassandra/Util.java:
##########
@@ -783,6 +784,13 @@ public static UnfilteredPartitionIterator
executeLocally(PartitionRangeReadComma
return command.queryStorage(cfs, controller);
}
+ public static UnfilteredPartitionIterator
executeLocally(SinglePartitionReadCommand command,
Review Comment:
thought about this more and I rather not change virtual tables in this
patch. I asked this question in slack (see
https://the-asf.slack.com/archives/CK23JSY2K/p1678145643633279) and it may make
sense to make public and depend on it, but would need to flesh out virtual
tables implementation as they don't use that method and will have failures if
touched.
--
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]
