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.

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

adding tx to queue needs to check gasprice in QKC #855

Merged
merged 2 commits into from
Mar 25, 2020
Merged

adding tx to queue needs to check gasprice in QKC #855

merged 2 commits into from
Mar 25, 2020

Conversation

qizhou
Copy link
Contributor

@qizhou qizhou commented Mar 24, 2020

This diff will enforce the minimum gasprice in QKC using converted QKC gasprice instead of gasprice of a native token directly. This avoids that a tx of a non-native token may be discarded since its gasprice is too small.

Following the fix, a couple of following things should be done:

  • More cluster tests with mnt
  • The transaction queue will order the txs by their gasprice in QKC instead of gasprice in their native token.

@qizhou qizhou requested review from hanyunx, ninjaahhh and a team March 24, 2020 00:03
quarkchain/cluster/shard_state.py Outdated Show resolved Hide resolved
quarkchain/cluster/shard_state.py Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants