Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Cross-Shard Congestion Control #539
Cross-Shard Congestion Control #539
Changes from 4 commits
81530e6
9262ddb
262d576
fc56f4c
41769c7
315952a
7af5edb
4c96103
5b018a9
690fef1
99ec21f
8d6cf6c
c3dd730
ec7528f
b82d806
e93d079
67fbc91
2f96c15
cd243a6
cb145a0
8029abf
3b9336c
3d7dea5
d2c6abf
f33e622
1e85668
fbb674c
8be43a3
8d1f88c
dc27a23
53c9d45
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have trouble following this process, because many variables (like gas_backlog, outgoing_gas, outgoing_gas_limit) are not previously well defined. Could we define these variables more carefully before this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, there were too many rounds of renaming things and in the end it was rather inconsistent. I completely missed that, thanks for pointing it out. It should be fixed now.
Check failure on line 149 in neps/nep-0539.md
GitHub Actions / markdown-lint
Fenced code blocks should be surrounded by blank lines [Context: "```python"]
Check failure on line 153 in neps/nep-0539.md
GitHub Actions / markdown-lint
Fenced code blocks should be surrounded by blank lines [Context: "```"]
Check failure on line 177 in neps/nep-0539.md
GitHub Actions / markdown-lint
Fenced code blocks should be surrounded by blank lines [Context: "```python"]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor detail: I know we use this pattern for
DELAYED_RECEIPT_OR_INDICES
, but it seems to be that way for historical reasons (see commit message here).For this new queue it would be clearer to have separate
BUFFERED_RECEIPT
andBUFFERED_RECEIPT_INDICES
columns.Check failure on line 212 in neps/nep-0539.md
GitHub Actions / markdown-lint
Emphasis used instead of a heading [Context: "TODO: describe in more details"]
Check failure on line 220 in neps/nep-0539.md
GitHub Actions / markdown-lint
Unordered list style [Expected: dash; Actual: asterisk]
Check failure on line 221 in neps/nep-0539.md
GitHub Actions / markdown-lint
Unordered list style [Expected: dash; Actual: asterisk]
Check failure on line 222 in neps/nep-0539.md
GitHub Actions / markdown-lint
Unordered list style [Expected: dash; Actual: asterisk]
Check failure on line 223 in neps/nep-0539.md
GitHub Actions / markdown-lint
Unordered list style [Expected: dash; Actual: asterisk]
Check failure on line 224 in neps/nep-0539.md
GitHub Actions / markdown-lint
Unordered list style [Expected: dash; Actual: asterisk]
Check failure on line 254 in neps/nep-0539.md
GitHub Actions / markdown-lint
Unordered list style [Expected: dash; Actual: asterisk]