Skip to content

Commit e2c7701

Browse files
committedNov 19, 2024
Remove old asserts
1 parent f845027 commit e2c7701

File tree

4 files changed

+3
-10
lines changed

4 files changed

+3
-10
lines changed
 

‎src/bucket/BucketListSnapshot.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,7 @@ SearchableBucketListSnapshot::loadPoolShareTrustLinesByAccountAndAsset(
263263
ZoneScoped;
264264

265265
// This query should only be called during TX apply
266-
releaseAssert(threadIsMain());
266+
// releaseAssert(threadIsMain());
267267
mSnapshotManager.maybeUpdateSnapshot(mSnapshot, mHistoricalSnapshots);
268268
releaseAssert(mSnapshot);
269269

‎src/bucket/BucketManagerImpl.cpp

+1-2
Original file line numberDiff line numberDiff line change
@@ -1020,7 +1020,7 @@ void
10201020
BucketManagerImpl::startBackgroundEvictionScan(uint32_t ledgerSeq,
10211021
bool callFromLedgerClose)
10221022
{
1023-
releaseAssert(!threadIsMain() || !mConfig.parallelLedgerClose());
1023+
// releaseAssert(!threadIsMain() || !mConfig.parallelLedgerClose());
10241024
releaseAssert(mConfig.isUsingBucketListDB());
10251025
releaseAssert(mSnapshotManager);
10261026
releaseAssert(!mEvictionFuture.valid());
@@ -1052,7 +1052,6 @@ BucketManagerImpl::resolveBackgroundEvictionScan(
10521052
LedgerKeySet const& modifiedKeys, SorobanNetworkConfig& networkConfig)
10531053
{
10541054
ZoneScoped;
1055-
releaseAssert(!threadIsMain() || !mConfig.parallelLedgerClose());
10561055
releaseAssert(mEvictionStatistics);
10571056

10581057
if (!mEvictionFuture.valid())

‎src/bucket/BucketSnapshotManager.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ BucketSnapshotManager::recordBulkLoadMetrics(std::string const& label,
4545
{
4646
// For now, only keep metrics for the main thread. We can decide on what
4747
// metrics make sense when more background services are added later.
48-
releaseAssert(threadIsMain());
48+
// releaseAssert(threadIsMain());
4949

5050
if (numEntries != 0)
5151
{

‎src/herder/TxSetFrame.cpp

-6
Original file line numberDiff line numberDiff line change
@@ -775,12 +775,6 @@ ApplicableTxSetFrame::ApplicableTxSetFrame(Application& app, bool isGeneralized,
775775
, mPhaseInclusionFeeMap(mTxPhases.size())
776776
, mContentsHash(contentsHash)
777777
{
778-
// print hashes
779-
CLOG_INFO(Herder, "Creating ApplicableTxSetFrame from txs: {}",
780-
hexAbbrev(previousLedgerHash));
781-
CLOG_INFO(
782-
Herder, "ApplicableTxSetFrame contents hash: {}",
783-
hexAbbrev(app.getLedgerManager().getLastClosedLedgerHeader().hash));
784778
releaseAssert(previousLedgerHash ==
785779
app.getLedgerManager().getLastClosedLedgerHeader().hash);
786780
}

0 commit comments

Comments
 (0)