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

Jump to bottom

vm: allow nonce gap for execution #3401

Closed
Tracked by #150
countvonzero opened this issue Aug 3, 2022 · 0 comments
Closed
Tracked by #150

vm: allow nonce gap for execution #3401

countvonzero opened this issue Aug 3, 2022 · 0 comments
Assignees
@countvonzero

Comments

@countvonzero
Copy link
Contributor

Description

see spacemeshos/pm#150

genvm/vm.go: change inequality (validity predicate) to compare >= NextNonce()
@countvonzero countvonzero added bug vm and removed bug labels Aug 3, 2022
This was referenced Aug 3, 2022
Closed
Closed
@countvonzero countvonzero self-assigned this Aug 4, 2022
bors bot pushed a commit that referenced this issue Aug 5, 2022
@countvonzero
allow nonce gap in mempool and exeuction (#3403)
ebcff3e 
## Motivation
<!-- Please mention the issue fixed by this PR or detailed motivation -->
Closes #3401 
Closes #3402
Closes #3389 

## Changes
<!-- Please describe in detail the changes made -->
- in vm relaxes the nonce comparison from `tx.Nonce == acct.NextNonce()` to `tx.Nonce >= acct.NextNonce()`
- in conservative cache
  - use container.List to keep mempool transactions
  - allow nonce gap

## Test Plan
<!-- Please specify how these changes were tested 
(e.g. unit tests, manual testing, etc.) -->
unit test, systest
bors bot pushed a commit that referenced this issue Aug 5, 2022
@countvonzero
allow nonce gap in mempool and exeuction (#3403)
0b4941d 
## Motivation
<!-- Please mention the issue fixed by this PR or detailed motivation -->
Closes #3401 
Closes #3402
Closes #3389 

## Changes
<!-- Please describe in detail the changes made -->
- in vm relaxes the nonce comparison from `tx.Nonce == acct.NextNonce()` to `tx.Nonce >= acct.NextNonce()`
- in conservative cache
  - use container.List to keep mempool transactions
  - allow nonce gap

## Test Plan
<!-- Please specify how these changes were tested 
(e.g. unit tests, manual testing, etc.) -->
unit test, systest
bors bot pushed a commit that referenced this issue Aug 5, 2022
@countvonzero
allow nonce gap in mempool and exeuction (#3403)
deacc76 
## Motivation
<!-- Please mention the issue fixed by this PR or detailed motivation -->
Closes #3401 
Closes #3402
Closes #3389 

## Changes
<!-- Please describe in detail the changes made -->
- in vm relaxes the nonce comparison from `tx.Nonce == acct.NextNonce()` to `tx.Nonce >= acct.NextNonce()`
- in conservative cache
  - use container.List to keep mempool transactions
  - allow nonce gap

## Test Plan
<!-- Please specify how these changes were tested 
(e.g. unit tests, manual testing, etc.) -->
unit test, systest
bors bot pushed a commit that referenced this issue Aug 5, 2022
@countvonzero
allow nonce gap in mempool and exeuction (#3403)
3fa5e41 
## Motivation
<!-- Please mention the issue fixed by this PR or detailed motivation -->
Closes #3401 
Closes #3402
Closes #3389 

## Changes
<!-- Please describe in detail the changes made -->
- in vm relaxes the nonce comparison from `tx.Nonce == acct.NextNonce()` to `tx.Nonce >= acct.NextNonce()`
- in conservative cache
  - use container.List to keep mempool transactions
  - allow nonce gap

## Test Plan
<!-- Please specify how these changes were tested 
(e.g. unit tests, manual testing, etc.) -->
unit test, systest
@bors bors bot closed this as completed in d4c11da Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@countvonzero countvonzero

Labels
None yet
Projects
No open projects
Dev iterations
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Merged by Bors] - allow nonce gap in mempool and exeuction spacemeshos/go-spacemesh
1 participant
@countvonzero