Skip to content

lib/runtime: implement ext_kill_child_storage and ext_get_allocated_child_storage#1110

Merged
edwardmack merged 3 commits intodevelopmentfrom
ed/ext_child_storage
Sep 30, 2020
Merged

lib/runtime: implement ext_kill_child_storage and ext_get_allocated_child_storage#1110
edwardmack merged 3 commits intodevelopmentfrom
ed/ext_child_storage

Conversation

@edwardmack
Copy link
Copy Markdown
Contributor

@edwardmack edwardmack commented Sep 28, 2020

Changes

  • Implement ext_kill_child_storage as defined in spec F.1.14
  • Implement ext_get_allocated_child_storage as defined in spec F.1,12
  • Update child storage prefix to :child_storage:default: as defined in spec F.1.9

Tests

go test ./lib/runtime/...

Checklist

  • I have read CODE_OF_CONDUCT and CONTRIBUTING
  • I have provided as much information as possible and necessary
  • I have reviewed my own pull request before requesting a review
  • All integration tests and required coverage checks are passing

Issues

@edwardmack edwardmack requested a review from noot September 28, 2020 22:21
@edwardmack edwardmack self-assigned this Sep 28, 2020

// ChildStorageKeyPrefix is the prefix for all child storage keys
var ChildStorageKeyPrefix = []byte(":child_storage:")
var ChildStorageKeyPrefix = []byte(":child_storage:default:")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where is this change from?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the spec, F.1.9 it says this key must be prefixed with :child_storage:default:

Copy link
Copy Markdown
Contributor

@noot noot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great work!

@edwardmack edwardmack merged commit 25393f3 into development Sep 30, 2020
@edwardmack edwardmack deleted the ed/ext_child_storage branch September 30, 2020 13:34
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