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

AVM: Share resource arrays across transactions #5035

Merged
merged 30 commits into from
Apr 7, 2023
Merged
Show file tree
Hide file tree
Changes from 28 commits
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
5051793
Initial implementation.
jannotti Jan 18, 2023
97e2269
Re-org availability checks in prep for changes
jannotti Jan 24, 2023
bbaa278
Checkpoint with new approach, needs v9 -> v8 tests
jannotti Jan 26, 2023
857d17e
Adjust names after rebase
jannotti Jan 26, 2023
bc2a5af
Add checks for inner txn resource access
jannotti Feb 1, 2023
726be12
Account for "shared" indexes in EvalDelta
jannotti Feb 2, 2023
a48886e
oas2 fixups
jannotti Feb 2, 2023
28aa602
e2e test for shared resource access
jannotti Feb 3, 2023
86dfb59
Several more tests, and handling created apps/accounts
jannotti Feb 15, 2023
e9dfd02
allocbound
jannotti Feb 16, 2023
bea0c92
coverage
jannotti Feb 16, 2023
ef20192
Test account passing across versions for more coverage
jannotti Feb 16, 2023
8b09e82
Bob CR
jannotti Feb 18, 2023
c70f0b6
Fix suggested by @jasonpaulos
jannotti Feb 21, 2023
b106666
Add (and use) a simpler way to do cross version app testing
jannotti Feb 22, 2023
d60380c
Fix app_opted_in for resource sharing, and initial spec changes
jannotti Feb 22, 2023
bf98ae4
Improvements from CR. Some more tests to come.
jannotti Feb 23, 2023
cf2bfeb
New tests, including v9 calling v8 showing blocked access
jannotti Feb 24, 2023
2bd96e1
Zeph CR typos
jannotti Mar 1, 2023
1ef746b
regenetate
jannotti Mar 20, 2023
8f738fb
Comment clarity
jannotti Mar 22, 2023
41f4f86
Pavel CR
jannotti Mar 23, 2023
56b2b55
Add tests for interpretation of integers as resource or slot
jannotti Mar 30, 2023
b4e63e8
Remove use of slots for app and asset indexes
jannotti Mar 30, 2023
946309e
Remove slot usage for account access opcodes
jannotti Mar 31, 2023
ccdd661
adjust ledger tests for v9
jannotti Apr 2, 2023
2239746
add extra test discussed with Pavel
jannotti Apr 3, 2023
3ad609f
Merge remote-tracking branch 'upstream/master' into avm-group-resources
jannotti Apr 4, 2023
0eb3923
fix long standing typo
jannotti Apr 4, 2023
530c76a
Final (?) cr fixes and an obscure bug fix
jannotti Apr 7, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions daemon/algod/api/algod.oas2.json
Original file line number Diff line number Diff line change
Expand Up @@ -2973,15 +2973,15 @@
"type": "integer"
},
"local-state-schema": {
"description": "[\\lsch\\] local schema",
"description": "\\[lsch\\] local schema",
"$ref": "#/definitions/ApplicationStateSchema"
},
"global-state-schema": {
"description": "[\\gsch\\] global schema",
"description": "\\[gsch\\] global schema",
"$ref": "#/definitions/ApplicationStateSchema"
},
"global-state": {
"description": "[\\gs\\] global schema",
"description": "\\[gs\\] global schema",
jannotti marked this conversation as resolved.
Show resolved Hide resolved
"$ref": "#/definitions/TealKeyValueStore"
}
}
Expand Down Expand Up @@ -3351,18 +3351,18 @@
"type": "integer"
},
"local-state-delta": {
"description": "\\[ld\\] Local state key/value changes for the application being executed by this transaction.",
"description": "Local state key/value changes for the application being executed by this transaction.",
"type": "array",
"items": {
"$ref": "#/definitions/AccountStateDelta"
}
},
"global-state-delta": {
"description": "\\[gd\\] Global state key/value changes for the application being executed by this transaction.",
"description": "Global state key/value changes for the application being executed by this transaction.",
"$ref": "#/definitions/StateDelta"
},
"logs": {
"description": "\\[lg\\] Logs for the application being executed by this transaction.",
"description": "Logs for the application being executed by this transaction.",
"type": "array",
"items": {
"type": "string",
Expand Down
4 changes: 2 additions & 2 deletions daemon/algod/api/algod.oas3.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1795,14 +1795,14 @@
"type": "array"
},
"local-state-delta": {
"description": "\\[ld\\] Local state key/value changes for the application being executed by this transaction.",
"description": "Local state key/value changes for the application being executed by this transaction.",
"items": {
"$ref": "#/components/schemas/AccountStateDelta"
},
"type": "array"
},
"logs": {
"description": "\\[lg\\] Logs for the application being executed by this transaction.",
"description": "Logs for the application being executed by this transaction.",
"items": {
"format": "byte",
"pattern": "^(?:[A-Za-z0-9+/]{4})*(?:[A-Za-z0-9+/]{2}==|[A-Za-z0-9+/]{3}=)?$",
Expand Down
73 changes: 36 additions & 37 deletions daemon/algod/api/server/v2/generated/data/routes.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

57 changes: 28 additions & 29 deletions daemon/algod/api/server/v2/generated/experimental/routes.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions daemon/algod/api/server/v2/generated/model/types.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading