LBRYCommunity/lbcd
1
0
Fork 0
Code Issues 17 Pull requests 4 Projects Releases 28 Packages Wiki Activity

Optimize duplicate transaction input check.

Browse source 
Profiling showed the duplicate transaction input check was taking around
6% of the total CheckTransactionSanity processing time.  This was largely
due to using fmt.Sprintf to generate the map key.

This commit modifies the check instead to use the actual output as a map
key.

The following benchmark results show the difference:

Before: BenchmarkOldDuplicatInputCheck    100000     21787 ns/op
After:  BenchmarkNewDuplicatInputCheck   2000000       937 ns/op

Closes #2
This commit is contained in:
Dave Collins 2013-10-26 14:31:15 -05:00
parent 89d86d07ac
commit b6e4ae4441
1 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
validate.go
View file

@ -230,14 +230,12 @@ func CheckTransactionSanity(tx *btcwire.MsgTx) error {
}
// Check for duplicate transaction inputs.
existingTxOut := make(map[string]bool)
existingTxOut := make(map[btcwire.OutPoint]bool)
for _, txIn := range tx.TxIn {
prevOut := &txIn.PreviousOutpoint
key := fmt.Sprintf("%v%v", prevOut.Hash, prevOut.Index)
if _, exists := existingTxOut[key]; exists {
if _, exists := existingTxOut[txIn.PreviousOutpoint]; exists {
return RuleError("transaction contains duplicate outpoint")
}
existingTxOut[key] = true
existingTxOut[txIn.PreviousOutpoint] = true
}
// Coinbase script length must be between min and max length.