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

Port more 0.10 changes #456

Merged
merged 4 commits into from
Oct 31, 2019
Merged

Conversation

arinwt
Copy link
Contributor

@arinwt arinwt commented Oct 31, 2019

Merges following PRs from 0.10 to master:

Arin Taylor added 4 commits October 31, 2019 12:06
* Handle errors during reloadContext and await pause/resume calls

* Always emit errors and reject promise on error

* Change reloadContext to reject throw error on catch
* Change SummaryManager to state machine

* Remove shouldNotStop telemetry event

* Clean up switch statement to prevent fallthrough

* Small changes for PR feedback

* Use finally for run completion
* Quorum ops count towards summarizer heuristics; summarizer handle ops while summarizing or pending

* Ignore summarizer leave messages, just in case

* Refactor summarizer code

* Add telemetry and small refactors

* Add comments; small refactor of summary op handling; track seq instead of flag
@arinwt arinwt merged commit 529fb19 into microsoft:master Oct 31, 2019
@arinwt arinwt deleted the port-arin-0.10-changes branch October 31, 2019 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants