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

feat: add prim deadline for receiver-side best-effort messages #4795

Merged
merged 19 commits into from
Dec 5, 2024

Conversation

ggreif
Copy link
Contributor

@ggreif ggreif commented Dec 3, 2024

Cf. ic0.msg_deadline in dfinity/portal#3764

Adds prim "deadline" and access to it via func replyDeadline in mo:prim.

All mainnet subnets are providing the system interface API, so this is safe to deploy.

Feature flag for drun is Enabled.

motoko-base support will be merged in short order.

@ggreif ggreif marked this pull request as draft December 3, 2024 23:27
@ggreif ggreif changed the title feat: add prim deadline for receiver side best-effort messages feat: add prim deadline for receiver-side best-effort messages Dec 4, 2024
@ggreif ggreif marked this pull request as ready for review December 4, 2024 00:12
Copy link

github-actions bot commented Dec 4, 2024

Comparing from 2873db9 to 4d9f34c:
In terms of gas, no changes are observed in 5 tests.
In terms of size, 5 tests regressed and the mean change is +0.0%.

@ggreif
Copy link
Contributor Author

ggreif commented Dec 4, 2024

@oggy-dfin Is the system interface API ic0.msg_deadline already present in all mainnet subnets?

@ggreif ggreif self-assigned this Dec 4, 2024
@ggreif ggreif added the feature New feature or request label Dec 4, 2024
rvanasa and others added 3 commits December 4, 2024 15:04
* Update readme

* Misc

* Unabbreviate 'OO'

* Try improving wording for OO line

* Update wording

* Adjust headers for consistency
@@ -26,10 +26,10 @@
"homepage": "",
"owner": "luc-blaeser",
"repo": "ic",
"rev": "33075cd385ea2ec4170f522406b1ff8f507fa744",
"sha256": "0swx7r78wjybx7fkfqsn9p3lqxmszgg24q4hrnxmscvyhb22hqia",
"rev": "bebe89514a6abd26e940b295323823169911a965",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

enables the ic0.msg_deadline in drun

crusso
crusso previously approved these changes Dec 5, 2024
Copy link
Contributor

@crusso crusso left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for getting started on this.

@crusso
Copy link
Contributor

crusso commented Dec 5, 2024

Might be worth testing some code in playground (uploading wasm) just to make sure its enabled on mainnet.

@crusso
Copy link
Contributor

crusso commented Dec 5, 2024

It wasn't clear from the interface spec PR if this signed or unsigned, but seems to be using u64 in ic source code...

@ggreif
Copy link
Contributor Author

ggreif commented Dec 5, 2024

Might be worth testing some code in playground (uploading wasm) just to make sure its enabled on mainnet.

https://play.motoko.org/?tag=661790008 has a deployed query2 canister. Some queries trap, but that is expected.

@ggreif ggreif added the automerge-squash When ready, merge (using squash) label Dec 5, 2024
@ggreif ggreif merged commit 491a820 into master Dec 5, 2024
8 checks passed
@ggreif ggreif deleted the gabor/deadline branch December 5, 2024 17:44
@mergify mergify bot removed the automerge-squash When ready, merge (using squash) label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants