swamirishi commented on code in PR #8214: URL: https://github.com/apache/ozone/pull/8214#discussion_r2069352809
########## hadoop-hdds/rocksdb-checkpoint-differ/src/main/java/org/apache/ozone/rocksdiff/RocksDBCheckpointDiffer.java: ########## @@ -163,14 +178,20 @@ public class RocksDBCheckpointDiffer implements AutoCloseable, private final long maxAllowedTimeInDag; private final BootstrapStateHandler.Lock lock = new BootstrapStateHandler.Lock(); - + private final StripedLockProvider omLock; + private static final String SST_FILE_LOCK = "SST_FILE_LOCK"; + private static final int SST_READ_AHEAD_SIZE = 2 * 1024 * 1024; + private int pruneSSTFileBatchSize; private ColumnFamilyHandle snapshotInfoTableCFHandle; private static final String DAG_PRUNING_SERVICE_NAME = "CompactionDagPruningService"; private AtomicBoolean suspended; private ColumnFamilyHandle compactionLogTableCFHandle; private ManagedRocksDB activeRocksDB; private ConcurrentMap<String, CompactionFileInfo> inflightCompactions; + private boolean isNativeLibsLoaded = false; + private BlockingQueue<Pair<byte[], CompactionLogEntry>> pruneQueue = Review Comment: Why BlockingQueue? Why not ConcurrentLinkedQueue? -- 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: issues-unsubscr...@ozone.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org For additional commands, e-mail: issues-h...@ozone.apache.org