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

fix: don't panic when the database is created by a higher version executable binary #2877

Merged
merged 2 commits into from
Jul 30, 2021

Conversation

yangby-cryptape
Copy link
Collaborator

@yangby-cryptape yangby-cryptape commented Jul 28, 2021

Commits

  • fix: don't panic when the database is created by a higher version executable binary

  • fix: the sub-command run shouldn't skip the fast migrations

    This bug doesn't affect any old versions because there is no fast migrations in old versions.

    Ref: What are fast migrations?.

@yangby-cryptape
Copy link
Collaborator Author

I will backport this PR to branch "rc/v0.43" after it merged.

@yangby-cryptape yangby-cryptape changed the title fix: several db issues fix: don't panic when the database is created by a higher version executable binary Jul 28, 2021
@yangby-cryptape yangby-cryptape marked this pull request as ready for review July 28, 2021 03:52
@yangby-cryptape yangby-cryptape requested a review from a team as a code owner July 28, 2021 03:52
@yangby-cryptape

This comment has been minimized.

@doitian
Copy link
Member

doitian commented Jul 30, 2021

bors r=doitian,driftluo,liya2017

@doitian doitian added the s:waiting-on-ci Status: Ready to merge, waiting for CI result label Jul 30, 2021
@bors
Copy link
Contributor

bors bot commented Jul 30, 2021

Build succeeded:

@bors bors bot merged commit 1744b27 into nervosnetwork:develop Jul 30, 2021
@yangby-cryptape yangby-cryptape deleted the pr/fix-db-issues branch July 30, 2021 10:37
@yangby-cryptape yangby-cryptape removed the s:waiting-on-ci Status: Ready to merge, waiting for CI result label Jul 30, 2021
bors bot added a commit that referenced this pull request Aug 2, 2021
2878: chore(hardfork): bump db migrate version for the new column family of block extension data r=doitian,quake a=yangby-cryptape

This PR is useless, unless the PR #2877 was backported to branch "rc/v0.43".

2889: ci: trigger ci on push specific branches r=liya2017,yangby-cryptape a=doitian

If we trigger on a branch and later skip the ci workflow, GitHub consider it as passed.

Side effect: forks cannot trigger this workflow other than the specific branches.

2890: chore: update README ci badges r=yangby-cryptape,quake a=doitian

* Specify branch explicitly
* Add query filter to the links


Co-authored-by: Boyu Yang <[email protected]>
Co-authored-by: ian <[email protected]>
@doitian doitian mentioned this pull request Aug 10, 2021
3 tasks
@doitian doitian mentioned this pull request Sep 3, 2021
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.

4 participants