Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

UB-1761 - Fixing concurrent map iteration and map write error #282

Merged
merged 1 commit into from
Dec 13, 2018

Conversation

deeghuge
Copy link
Member

@deeghuge deeghuge commented Dec 12, 2018

Fixing error concurrent map iteration and map write error


This change is Reviewable

@deeghuge deeghuge changed the title UB-1761 - Fixing error concurrent map iteration and map write error UB-1761 - Fixing concurrent map iteration and map write error Dec 12, 2018
@coveralls
Copy link

coveralls commented Dec 12, 2018

Coverage Status

Coverage remained the same at 58.774% when pulling a52b99f on UB-1761 into ae2f7c4 on release_v2.0.0.

Copy link
Collaborator

@olgashtivelman olgashtivelman left a comment

Choose a reason for hiding this comment

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

:lgtm:

just FYI you are merging this to dev but this should also be merged to the release branch : release_v2.0.0.

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ranhrl)

Copy link
Collaborator

@ranhrl ranhrl left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ranhrl)

@deeghuge deeghuge closed this Dec 12, 2018
@deeghuge deeghuge reopened this Dec 12, 2018
@deeghuge deeghuge changed the base branch from dev to release_v2.0.0 December 12, 2018 14:19
@olgashtivelman olgashtivelman merged commit 81950e3 into release_v2.0.0 Dec 13, 2018
olgashtivelman pushed a commit that referenced this pull request Dec 13, 2018
…ting map at same time. (#282)

Fixing error concurrent map iteration and map write error
@deeghuge deeghuge deleted the UB-1761 branch December 19, 2018 17:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants