blambov commented on code in PR #2065:
URL: https://github.com/apache/cassandra/pull/2065#discussion_r1067855935
##########
src/java/org/apache/cassandra/db/rows/UnfilteredRowIteratorWithLowerBound.java:
##########
@@ -186,7 +193,7 @@ private static <V> ClusteringBound<V>
createArtificialLowerBound(boolean isRever
/**
* @return the lower bound stored on the index entry for this partition,
if available.
*/
- private ClusteringBound<?> getPartitionIndexLowerBound()
+ private ClusteringBound<?> getKeyCacheLowerBound()
{
// NOTE: CASSANDRA-11206 removed the lookup against the key-cache as
the IndexInfo objects are no longer
Review Comment:
I believe this comment is not correct (row indexes can also reside on the
heap, if they are small enough, and the `rowIndexEntry.indexOnHeap()` call
below checks this).
--
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]
