maedhroz commented on code in PR #2267:
URL: https://github.com/apache/cassandra/pull/2267#discussion_r1164440372
##########
src/java/org/apache/cassandra/io/sstable/format/big/BigTableScanner.java:
##########
@@ -345,11 +339,15 @@ protected UnfilteredRowIterator computeNext()
/*
* For a given partition key, we want to avoid hitting the data
- * file unless we're explicitely asked to. This is important
+ * file unless we're explicitly asked to. This is important
* for PartitionRangeReadCommand#checkCacheFilter.
*/
return new LazilyInitializedUnfilteredRowIterator(currentKey)
{
+ // Store currentEntry reference during object
instantiation as later (during initialize) the
+ // reference may point to a different entry.
+ private final RowIndexEntry rowIndexEntry = currentEntry;
Review Comment:
Was there are test failing somewhere around this, or is it just something we
noticed offhand?
--
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]
