mirror of https://github.com/apache/cassandra
Remove unused filterSSTablesForReads
Patch by marcuse; reviewed by Jeremiah Jordan for CASSANDRA-9553
This commit is contained in:
parent
eb266ce8d9
commit
4a61ca6878
|
|
@ -1833,7 +1833,7 @@ public class ColumnFamilyStore implements ColumnFamilyStoreMBean
|
|||
{
|
||||
public List<SSTableReader> apply(View view)
|
||||
{
|
||||
return compactionStrategyManager.filterSSTablesForReads(view.intervalTree.search(key));
|
||||
return view.intervalTree.search(key);
|
||||
}
|
||||
};
|
||||
}
|
||||
|
|
@ -1848,7 +1848,7 @@ public class ColumnFamilyStore implements ColumnFamilyStoreMBean
|
|||
{
|
||||
public List<SSTableReader> apply(View view)
|
||||
{
|
||||
return compactionStrategyManager.filterSSTablesForReads(view.sstablesInBounds(rowBounds));
|
||||
return view.sstablesInBounds(rowBounds);
|
||||
}
|
||||
};
|
||||
}
|
||||
|
|
|
|||
|
|
@ -237,14 +237,6 @@ public abstract class AbstractCompactionStrategy
|
|||
CompactionManager.instance.submitBackground(cfs);
|
||||
}
|
||||
|
||||
/**
|
||||
* @return a subset of the suggested sstables that are relevant for read requests.
|
||||
*/
|
||||
public List<SSTableReader> filterSSTablesForReads(List<SSTableReader> sstables)
|
||||
{
|
||||
return sstables;
|
||||
}
|
||||
|
||||
/**
|
||||
* Filters SSTables that are to be blacklisted from the given collection
|
||||
*
|
||||
|
|
|
|||
|
|
@ -205,15 +205,6 @@ public class CompactionStrategyManager implements INotificationConsumer
|
|||
CompactionManager.instance.submitBackground(cfs);
|
||||
}
|
||||
|
||||
/**
|
||||
* TODO: remove, unused
|
||||
*/
|
||||
public List<SSTableReader> filterSSTablesForReads(List<SSTableReader> sstables)
|
||||
{
|
||||
// todo: union of filtered sstables or intersection?
|
||||
return unrepaired.filterSSTablesForReads(repaired.filterSSTablesForReads(sstables));
|
||||
}
|
||||
|
||||
public int getUnleveledSSTables()
|
||||
{
|
||||
if (repaired instanceof LeveledCompactionStrategy && unrepaired instanceof LeveledCompactionStrategy)
|
||||
|
|
|
|||
Loading…
Reference in New Issue