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

stream: remove semaphor hack #2096

Merged
merged 1 commit into from
Feb 7, 2020
Merged

stream: remove semaphor hack #2096

merged 1 commit into from
Feb 7, 2020

Conversation

acud
Copy link
Member

@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 self-assigned this Feb 7, 2020
@acud acud added this to Backlog in Swarm Core - Sprint planning via automation Feb 7, 2020
@acud acud moved this from Backlog to In review (includes Documentation) in Swarm Core - Sprint planning Feb 7, 2020
@acud acud merged commit a836208 into master Feb 7, 2020
Swarm Core - Sprint planning automation moved this from In review (includes Documentation) to Done 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
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants