jacek-lewandowski commented on code in PR #2065:
URL: https://github.com/apache/cassandra/pull/2065#discussion_r1067793853
##########
src/java/org/apache/cassandra/db/rows/UnfilteredRowIteratorWithLowerBound.java:
##########
@@ -185,70 +183,8 @@ public Row staticRow()
return super.staticRow();
}
- private static <V> ClusteringBound<V> createArtificialLowerBound(boolean
isReversed, ClusteringPrefix<V> from)
- {
- return from.accessor().factory().inclusiveOpen(isReversed,
from.getRawValues()).artificialLowerBound(isReversed);
- }
-
- /**
- * @return the lower bound stored on the index entry for this partition,
if available.
- */
- private ClusteringBound<?> getKeyCacheLowerBound()
- {
- // NOTE: CASSANDRA-11206 removed the lookup against the key-cache as
the IndexInfo objects are no longer
- // in memory for not heap backed IndexInfo objects (so, these are on
disk).
- // CASSANDRA-11369 is there to fix this afterwards.
-
- // Creating the iterator ensures that rowIndexEntry is loaded if
available (partitions bigger than
- // DatabaseDescriptor.column_index_size)
- if (!canUseMetadataLowerBound())
- maybeInit();
-
- RowIndexEntry rowIndexEntry =
sstable.getCachedPosition(partitionKey(), false);
Review Comment:
Not you, the comment:
```
// Creating the iterator ensures that rowIndexEntry is loaded if
available (partitions bigger than
// DatabaseDescriptor.column_index_size)
if (!canUseMetadataLowerBound())
maybeInit();
```
--
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]
