Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Tx Queue improvements #2292

Merged
merged 9 commits into from
Sep 26, 2016
Merged

Tx Queue improvements #2292

merged 9 commits into from
Sep 26, 2016

Conversation

tomusdrw
Copy link
Collaborator

  1. Better logs (debug)
  2. Transaction queue content (pending) accessible over RPC (very basic)
  3. Deprioritizing transactions that exceeds current gas limit.

Tomasz Drwięga and others added 7 commits September 23, 2016 20:30
Improving txqueue logs
62cbf9c
Penalize transactions with gas above gas limit
2874f46 
Conflicts:
	ethcore/src/miner/transaction_queue.rs
Avoid penalizing legit transactions
efaef85
@tomusdrw
Peek transaction queue via RPC (#2270)
3e47421 
* Handle RLP to string UTF-8 decoding errors (#2217)

* pending transactions

Conflicts:
	rpc/src/v1/impls/ethcore.rs
	rpc/src/v1/traits/ethcore.rs
@gavofyork
saturating not overflowing
5e7955b
@gavofyork
Remove crufty code
06fc494
saturating add
97fcd60 
Conflicts:
	ethcore/src/miner/transaction_queue.rs
@tomusdrw tomusdrw added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Sep 23, 2016
@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. and removed A0-pleasereview 🤓 Pull request needs code review. A8-looksgood 🦄 Pull request is reviewed well. labels Sep 25, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.007%) to 85.22% when pulling 4acd6cb on txqueue into abcdc81 on master.

@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. and removed A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. labels Sep 26, 2016
@rphmeier rphmeier merged commit fe4f103 into master Sep 26, 2016
@arkpar arkpar deleted the txqueue branch October 3, 2016 15:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tomusdrw @coveralls @gavofyork @rphmeier