diff options
author | Jimmy Hu <jimmy.hu@dexon.org> | 2018-11-21 10:26:36 +0800 |
---|---|---|
committer | Wei-Ning Huang <w@dexon.org> | 2018-12-19 20:54:27 +0800 |
commit | 2cf60ac04074e08bf5f33d72c763e988ea4db82c (patch) | |
tree | 63572759a0ce1a7218b9484514d89a7d32d80bb5 | |
parent | 527f41661d7fdfa23abe1402847cc8c7da6e8e12 (diff) | |
download | dexon-2cf60ac04074e08bf5f33d72c763e988ea4db82c.tar dexon-2cf60ac04074e08bf5f33d72c763e988ea4db82c.tar.gz dexon-2cf60ac04074e08bf5f33d72c763e988ea4db82c.tar.bz2 dexon-2cf60ac04074e08bf5f33d72c763e988ea4db82c.tar.lz dexon-2cf60ac04074e08bf5f33d72c763e988ea4db82c.tar.xz dexon-2cf60ac04074e08bf5f33d72c763e988ea4db82c.tar.zst dexon-2cf60ac04074e08bf5f33d72c763e988ea4db82c.zip |
core, dex: Batch process touchSender. Lower priority for tx. (#41)
* dex: Add a tx queue in broadcast
* Modify queue parameter
* Priority select all messages except tx
* Batch process TouchSenders
-rw-r--r-- | core/types/transaction.go | 32 | ||||
-rw-r--r-- | dex/handler.go | 1 |
2 files changed, 20 insertions, 13 deletions
diff --git a/core/types/transaction.go b/core/types/transaction.go index 63195f4fd..6ee6e4494 100644 --- a/core/types/transaction.go +++ b/core/types/transaction.go @@ -21,6 +21,7 @@ import ( "errors" "io" "math/big" + "runtime" "sync" "sync/atomic" @@ -274,24 +275,29 @@ func (s Transactions) GetRlp(i int) []byte { // TouchSenders calculates the sender of each transaction and update the cache. func (s Transactions) TouchSenders(signer Signer) (errorTx *Transaction, err error) { + num := runtime.NumCPU() + batchSize := len(s) / num wg := sync.WaitGroup{} - wg.Add(len(s)) + wg.Add(num) txError := make(chan error, 1) - for _, tx := range s { - go func(tx *Transaction) { + for i := 0; i < num; i++ { + go func(txs Transactions) { defer wg.Done() - if len(txError) > 0 { - return - } - _, err := Sender(signer, tx) - if err != nil { - select { - case txError <- err: - errorTx = tx - default: + for _, tx := range txs { + if len(txError) > 0 { + return + } + _, err := Sender(signer, tx) + if err != nil { + select { + case txError <- err: + errorTx = tx + default: + } + return } } - }(tx) + }(s[i*batchSize : (i+1)*batchSize]) } wg.Wait() select { diff --git a/dex/handler.go b/dex/handler.go index 8ab206d33..6b2e3271b 100644 --- a/dex/handler.go +++ b/dex/handler.go @@ -729,6 +729,7 @@ func (pm *ProtocolManager) handleMsg(p *peer) error { } p.MarkTransaction(tx.Hash()) } + types.Transactions(txs).TouchSenders(types.MakeSigner(pm.blockchain.Config(), new(big.Int))) pm.txpool.AddRemotes(txs) case msg.Code == MetaMsg: |