blambov commented on code in PR #2267:
URL: https://github.com/apache/cassandra/pull/2267#discussion_r1171597824
##########
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:
Test for the associated failure plus proper fix added.
--
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]
