Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

metrics: split out /Transaction/AssembleBlock metrics #4795

Merged
merged 6 commits into from
Nov 17, 2022
Merged

metrics: split out /Transaction/AssembleBlock metrics #4795

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
e69f926
Split out logging on invalid transactions in /Transaction/AssembleBlo…
iansuvak Nov 15, 2022
c17d3f4
add an explicit counter for transactions expired with LastValid set t…
iansuvak Nov 15, 2022
d2e08bc
simplify the type assertion in switch
iansuvak Nov 15, 2022
b6b079f
Address review feedback
iansuvak Nov 16, 2022
57e08ac
add log info back for LeaseInLedgerError
iansuvak Nov 17, 2022
3560a26
address feedback
iansuvak Nov 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
address feedback
  • Loading branch information
@iansuvak
iansuvak committed Nov 17, 2022
commit 3560a26f60e222f857f037da396664296b617641
4 changes: 2 additions & 2 deletions ledger/ledgercore/error.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,8 @@ func MakeLeaseInLedgerError(txid transactions.Txid, lease Txlease) *LeaseInLedge
// Error implements the error interface for the LeaseInLedgerError stuct
func (lile *LeaseInLedgerError) Error() string {
// format the lease as address.
addr := basics.Address(lile.lease.Lease)
return fmt.Sprintf("transaction %v using an overlapping lease (%s, %s)", lile.txid, lile.lease.Sender.String(), addr.String())
leaseValue := basics.Address(lile.lease.Lease)
return fmt.Sprintf("transaction %v using an overlapping lease (sender, lease):(%s, %s)", lile.txid, lile.lease.Sender.String(), leaseValue.String())
}

// BlockInLedgerError is returned when a block cannot be added because it has already been done
Expand Down