Skip to content

Commit

Permalink
Add logging of pointer SeqIterator to initBySeqnoScanContext()
Browse files Browse the repository at this point in the history
Log the pointer's address for the new SeqIterator created by
MagmaKVStore::initBySeqnoScanContext so that we know which SeqIterator
is being used for which stream, as we only log the SeqIterator's pointer
inside BySeqIterator::Seek().

Change-Id: I859fc66d566ebfbd229ca60ac3071eb221b28f06
Reviewed-on: https://review.couchbase.org/c/kv_engine/+/168545
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>
  • Loading branch information
rdemellow authored and daverigby committed Jan 12, 2022
1 parent 36930c5 commit 0335e19
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions engines/ep/src/kvstore/magma-kvstore/magma-kvstore.cc
Expand Up @@ -1637,14 +1637,16 @@ std::unique_ptr<BySeqnoScanContext> MagmaKVStore::initBySeqnoScanContext(

logger->info(
"MagmaKVStore::initBySeqnoScanContext {} seqno:{} endSeqno:{}"
" purgeSeqno:{} nDocsToRead:{} docFilter:{} valFilter:{}",
" purgeSeqno:{} nDocsToRead:{} docFilter:{} valFilter:{} "
"SeqIterator:{:p}",
vbid,
startSeqno,
highSeqno,
purgeSeqno,
nDocsToRead,
docFilter,
valFilter);
valFilter,
fmt::ptr(itr.get()));
}

auto mctx = std::make_unique<MagmaScanContext>(std::move(cb),
Expand Down

0 comments on commit 0335e19

Please sign in to comment.