Skip to content

pin: fix concurrent map access race#6419

Merged
Stebalien merged 1 commit intomasterfrom
fix/6418
Jun 6, 2019
Merged

pin: fix concurrent map access race#6419
Stebalien merged 1 commit intomasterfrom
fix/6418

Conversation

@Stebalien
Copy link
Copy Markdown
Member

Not sure why this didn't show up sooner.

fixes #6418

Not sure why this didn't show up sooner.

fixes #6418
@Stebalien
Copy link
Copy Markdown
Member Author

cc @postables for review.

Copy link
Copy Markdown

@bonedaddy bonedaddy left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Stebalien Stebalien merged commit c7dc3c3 into master Jun 6, 2019
@Stebalien Stebalien deleted the fix/6418 branch June 6, 2019 22:50
@zhjgit
Copy link
Copy Markdown

zhjgit commented Jun 27, 2019

#6418 After which version, please fix the bug. @Stebalien @postables I'm currently using go-ipfs 0.4.18 and this problem has arisen.

@Stebalien
Copy link
Copy Markdown
Member Author

We need to cut a release for this.

@zhjgit
Copy link
Copy Markdown

zhjgit commented Jun 27, 2019

@Stebalien When will go-ipfs 0.4.18 be released to fix the bug? I need to use it urgently.

@Stebalien
Copy link
Copy Markdown
Member Author

Please build the latest master. We're planning on cutting an RC for the next release ASAP (days) but I can't give an exact date.

@zhjgit
Copy link
Copy Markdown

zhjgit commented Jun 27, 2019

I can't use the latest version. I only need version 0.4.18

@Stebalien
Copy link
Copy Markdown
Member Author

We will not backport to 0.4.18. What's your issue with the latest version?

@zhjgit
Copy link
Copy Markdown

zhjgit commented Jun 28, 2019

That's the problem. #6418

Stebalien added a commit that referenced this pull request Jul 4, 2019
pin: fix concurrent map access race
@Stebalien Stebalien mentioned this pull request Jul 12, 2019
51 tasks
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.

fatal error: concurrent map iteration and map write

3 participants