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

Queue RPCs by Worker_EntityId #1138

Merged
merged 5 commits into from
Jul 9, 2019

Conversation

aleximprobable
Copy link
Contributor

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.


Description

RPCs are now guaranteed to arrive in the same order per target actor, i.e. for the actor and all of its subobjects.

Release note

RPCs are now guaranteed to arrive in the same order per target actor, i.e. for the actor and all of its subobjects.

Tests

RPC ordering should be the same as in original UE4 implementation.

Documentation

RPCs are now guaranteed to arrive in the same order per target actor, i.e. for the actor and all of its subobjects.

Primary reviewers

@improbable-valentyn @m-samiec

Co-Authored-By: improbable-valentyn <32096431+improbable-valentyn@users.noreply.github.com>
Copy link
Collaborator

@m-samiec m-samiec left a comment

Choose a reason for hiding this comment

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

Couple of drivebys.

@aleximprobable aleximprobable merged commit 9396e73 into master Jul 9, 2019
@aleximprobable aleximprobable deleted the bugfix/unr-1663-per-actor-ordering-guarantee branch July 9, 2019 14:27
improbable-valy pushed a commit that referenced this pull request Jul 10, 2019
* Queue RPCs by Worker_EntityId

* Apply suggestions from code review

Co-Authored-By: improbable-valentyn <32096431+improbable-valentyn@users.noreply.github.com>

* Changed Worker_EntityId to Worker_EntityId_Key for Linux compilation. Fixed function signature

* Moved initialisation inside if statements
improbable-valy added a commit that referenced this pull request Jul 10, 2019
* Queue RPCs by Worker_EntityId

* Apply suggestions from code review

Co-Authored-By: improbable-valentyn <32096431+improbable-valentyn@users.noreply.github.com>

* Changed Worker_EntityId to Worker_EntityId_Key for Linux compilation. Fixed function signature

* Moved initialisation inside if statements
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.

3 participants