Only warn on non-pseudo transactions without accounts:

Closes #3153
This commit is contained in:
Scott Schurr
2021-04-19 14:45:44 -07:00
committed by manojsdoshi
parent 2a7c573dec
commit dcba79be48
2 changed files with 8 additions and 4 deletions

View File

@@ -317,12 +317,14 @@ saveValidatedLedger(
JLOG(j.trace()) << "ActTx: " << sql;
*db << sql;
}
else
else if (auto const sleTxn = acceptedLedgerTx->getTxn();
!isPseudoTx(*sleTxn))
{
// It's okay for pseudo transactions to not affect any
// accounts. But otherwise...
JLOG(j.warn()) << "Transaction in ledger " << seq
<< " affects no accounts";
JLOG(j.warn()) << acceptedLedgerTx->getTxn()->getJson(
JsonOptions::none);
JLOG(j.warn()) << sleTxn->getJson(JsonOptions::none);
}
*db

View File

@@ -274,8 +274,10 @@ updateLedgerDBs(
JLOG(j.trace())
<< "shard " << index << " account transaction: " << sql;
}
else
else if (!isPseudoTx(*item.first))
{
// It's okay for pseudo transactions to not affect any
// accounts. But otherwise...
JLOG(j.warn())
<< "shard " << index << " transaction in ledger "
<< sSeq << " affects no accounts";