jacek-lewandowski commented on code in PR #2064:
URL: https://github.com/apache/cassandra/pull/2064#discussion_r1072137813
##########
src/java/org/apache/cassandra/index/sasi/SASIIndexBuilder.java:
##########
@@ -56,22 +55,23 @@ class SASIIndexBuilder extends SecondaryIndexBuilder
private final SortedMap<SSTableReader, Map<ColumnMetadata, ColumnIndex>>
sstables;
private long bytesProcessed = 0;
- private final long totalSizeInBytes;
+ private final long totalBytesToProcess;
public SASIIndexBuilder(ColumnFamilyStore cfs, SortedMap<SSTableReader,
Map<ColumnMetadata, ColumnIndex>> sstables)
{
- long totalIndexBytes = 0;
+ long totalBytesToProcess = 0;
for (SSTableReader sstable : sstables.keySet())
- totalIndexBytes += getPrimaryIndexLength(sstable);
+ totalBytesToProcess += sstable.uncompressedLength();
Review Comment:
I don't know where, the existing usages of `CompactionInfo` look like they
already assumed the data size rather than the index size.
--
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]
