Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

feat(sync-call): [IC-1666] Endpoint for synchronous call requests #265

Merged
merged 50 commits into from
Sep 12, 2024

Conversation

DSharifi
Copy link
Member

@DSharifi DSharifi commented Jan 11, 2024

As part of IC-1666 the proposal introduces a new HTTPS endpoint for synchronous update calls, or "call and await", where users can send call requests to the replica and get the response back in the same request. This differs from the current call approach where users have to continuously poll the status of a call request to get the response.

Adding a synchronous endpoint for update calls will significantly decrease the end-to-end (client-observed) latency for update calls. An initial prototype shows that the client-observed latency can be reduced by a minimum of 1 second which is ~40% improvement.

@DSharifi DSharifi changed the title draft: Sync update call Endpoint for synchronous update calls Jan 11, 2024
@DSharifi DSharifi marked this pull request as ready for review January 11, 2024 16:33
@DSharifi DSharifi requested a review from a team as a code owner January 11, 2024 16:33
@DSharifi DSharifi self-assigned this Jan 11, 2024
@DSharifi DSharifi requested a review from rumenov January 11, 2024 16:34
@DSharifi DSharifi changed the title Endpoint for synchronous update calls feat(sync-call): [IC-1666] Endpoint for synchronous call requests Jan 11, 2024
@DSharifi DSharifi marked this pull request as draft January 11, 2024 16:49
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
Co-authored-by: mraszyk <31483726+mraszyk@users.noreply.github.com>
@DSharifi DSharifi requested a review from yotamhc January 12, 2024 08:20
spec/index.md Outdated Show resolved Hide resolved
@DSharifi DSharifi requested a review from mraszyk April 18, 2024 07:57
@DSharifi DSharifi marked this pull request as ready for review April 18, 2024 07:58
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
@mraszyk mraszyk merged commit c28afe4 into master Sep 12, 2024
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants