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

vendor: update boltdb #10808

Merged
merged 1 commit into from
Jun 10, 2019
Merged

vendor: update boltdb #10808

merged 1 commit into from
Jun 10, 2019

Conversation

WIZARD-CXY
Copy link
Contributor

@WIZARD-CXY WIZARD-CXY commented Jun 10, 2019

update the boltdb dependency to v1.3.3. mainly solve the panic bug in boltdb @xiang90 @jingyih

@xiang90
Copy link
Contributor

xiang90 commented Jun 10, 2019

lgtm

1 similar comment
@jingyih
Copy link
Contributor

jingyih commented Jun 10, 2019

lgtm

@jingyih jingyih merged commit 833620b into etcd-io:master Jun 10, 2019
@jingyih
Copy link
Contributor

jingyih commented Jun 10, 2019

I will update the changelog accordingly.

@jingyih
Copy link
Contributor

jingyih commented Jun 10, 2019

Do we want to backport this? Is the bug "rollback panic when freelist nosync is enabled" only in master?

jingyih added a commit that referenced this pull request Jun 10, 2019
@WIZARD-CXY
Copy link
Contributor Author

@jingyih I think we should backport it

@jingyih
Copy link
Contributor

jingyih commented Jun 10, 2019

@jingyih I think we should backport it

@WIZARD-CXY Thanks. Do you have bandwidth to help back porting? If not, I can do it tomorrow.

@WIZARD-CXY
Copy link
Contributor Author

@jingyih I think etcd 3.3 involves the nosync freelist feature. we can backport it to only v3.3 if I think it right.

@jingyih
Copy link
Contributor

jingyih commented Jun 11, 2019

If the bug only appears after 870302a, then only 3.3 has it.

@WIZARD-CXY
Copy link
Contributor Author

yes

@jingyih
Copy link
Contributor

jingyih commented Aug 20, 2019

For reference, bbolt is upgraded to v1.3.3 in 3.3 branch via #11056.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants