Skip to content
This repository was archived by the owner on Aug 2, 2021. It is now read-only.

stream: remove semaphor hack#2096

Merged
acud merged 1 commit intomasterfrom
stream-rm-sem-hack
Feb 7, 2020
Merged

stream: remove semaphor hack#2096
acud merged 1 commit intomasterfrom
stream-rm-sem-hack

Conversation

@acud
Copy link
Copy Markdown
Contributor

@acud acud commented Feb 7, 2020

I would like to revert this change since it has caused a significant instability in tests (many tests are flaking because of this change). I am not sure if we actually get this change to negatively affect syncing on production environment, but to be on the conservative side I'd prefer this to be removed.

@acud acud added the cleanup code completion, add comments and more label Feb 7, 2020
@acud acud added this to the 0.5.6 milestone Feb 7, 2020
@acud acud requested review from janos, pradovic, svetomir and zelig February 7, 2020 15:06
@acud acud self-assigned this Feb 7, 2020
@acud acud force-pushed the stream-rm-sem-hack branch from b2004db to 7fa2005 Compare February 7, 2020 15:32
@acud acud merged commit a836208 into master Feb 7, 2020
@acud acud deleted the stream-rm-sem-hack branch February 7, 2020 16:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cleanup code completion, add comments and more

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants