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

EndpointRegistry fixes #1862

Merged
merged 1 commit into from
Apr 12, 2016
Merged

Conversation

Aaronontheweb
Copy link
Member

Fixed numerous issues with the EndpointRegistry and added specs to cover - should resolve #1845 and others.

@@ -193,9 +193,6 @@ EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "PersistenceBenchmark", "benchmark\PersistenceBenchmark\PersistenceBenchmark.csproj", "{39E6F51F-FA1E-4C62-B8F8-19065DE6D55D}"
EndProject
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Solution Items", "Solution Items", "{A59BAE84-70E2-46A0-9E26-7413C103E2D7}"
ProjectSection(SolutionItems) = preProject
Copy link
Member Author

Choose a reason for hiding this comment

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

No idea what this was for. Deleted it.

@Aaronontheweb Aaronontheweb changed the title [WIP] Fix endpoint registry EndpointRegistry fixes Apr 7, 2016
@Aaronontheweb
Copy link
Member Author

This one is ready to review cc @annymsMthd

@Aaronontheweb
Copy link
Member Author

Hrmmm.... hold the phone - found another potential issue that could be part of #1700

@Aaronontheweb
Copy link
Member Author

Ok, did a pretty full review of the diff on #1755 - only thing I could come up with was that we had two bugs that offset each other and I only fixed one of them in 1.0.6. One bug had us pass in the wrong UID to the ProtocolStateActor all the way down from the EndpointManager and the second bug was falsely reporting that we needed a read-only endpoint, which resulted in the ReliableDeliverySupervisor not being started correctly.

This PR fixes the second bug, which was caused by an issue with the EndpointRegistry. Looking at including some more MNTR specs along with this...

@maxim-s
Copy link
Contributor

maxim-s commented Apr 8, 2016

Great work

@maxim-s maxim-s merged commit 30ecf5f into akkadotnet:dev Apr 12, 2016
@Aaronontheweb Aaronontheweb deleted the fix-endpoint-registry branch April 12, 2016 15:05
This was referenced Apr 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unexpected(?) exception in EndpointRegistry.RegisterReadOnlyEndpoint()
2 participants