-
Notifications
You must be signed in to change notification settings - Fork 1k
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
RemoteActorRefProvider address paring, caching and resolving improvements #5273
Merged
Aaronontheweb
merged 41 commits into
akkadotnet:dev
from
Zetanova:perf-remote-actorref-provider
Oct 1, 2021
Merged
Changes from 1 commit
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
f25de15
refactor remote-actorref-provider and add tests for cache entries
Zetanova 3fc0168
replace address-cache with actorpath-cache
Zetanova 9a67f1c
refactor resolve with local address
Zetanova 746f76b
refactor and cleanup
Zetanova bff01bb
remove volatile from fields
Zetanova d0f232e
Merge branch 'dev' into perf-remote-actorref-provider
Zetanova 3dc67a5
merge upstream
Zetanova b458ae4
Merge branch 'perf-remote-actorref-provider' of https://github.com/Ze…
Zetanova 2a55502
remove double equals
Zetanova 14e88e2
cleanup
Zetanova 762ec30
refactor to base
Zetanova aea2166
optimize elements list
Zetanova a36170f
improve actor path join
Zetanova aa211c7
improve actor path equals and compare
Zetanova 1842505
cleanup
Zetanova f388cef
protect stack and use moveto of arraybuilder
Zetanova baaeea2
update api spec
Zetanova f86dc4b
test for jumbo actor path name support
Zetanova 4427087
small refactors
Zetanova 93f5d9c
add ActorPath.ParentOf(depth)
Zetanova d74deaa
dont copy actorpath
Zetanova a7a525e
use actorpath-cache and remove cache entry test
Zetanova d3c33e8
refactor fill array
Zetanova 4512aec
prepair actor path cache for better deduplication
Zetanova ad3ca76
update api
Zetanova 026a6fc
cache root actor path
Zetanova b29e242
update api
Zetanova 9d9a2d7
remove obsolete code
Zetanova 29eaff2
cleanup code
Zetanova f33929e
Merge branch 'dev' into perf-remote-actorref-provider
Aaronontheweb 1051ed3
Merge remote-tracking branch 'upstream/dev' into perf-remote-actorref…
Zetanova 2564415
Merge branch 'perf-remote-actorref-provider' of https://github.com/Ze…
Zetanova d3a0ae0
removed commented cache tests
Zetanova 217485c
refactor span to string bulder
Zetanova 409485f
use internal fields and ref equals
Zetanova 9eba407
add rebase path test
Zetanova cc30aa1
fix possible NRE
Zetanova 73d26ff
extend and test address parsing
Zetanova c13ad50
update api
Zetanova 793b8b5
Merge branch 'dev' into perf-remote-actorref-provider
Zetanova 8dd6e0c
Merge branch 'dev' into perf-remote-actorref-provider
Aaronontheweb File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
optimize elements list
- Loading branch information
commit aea216692a91126108d589e181c51bab2fff477c
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM