Skip to content

Commit

Permalink
bump snapshot version
Browse files Browse the repository at this point in the history
  • Loading branch information
tbfleming committed Jan 15, 2022
1 parent 81be25f commit 768d61f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion libraries/chain/include/eosio/chain/chain_snapshot.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ struct chain_snapshot_header {
*/

static constexpr uint32_t minimum_compatible_version = 2;
static constexpr uint32_t current_version = 4;
static constexpr uint32_t current_version = 5;

uint32_t version = current_version;

Expand Down
2 changes: 1 addition & 1 deletion unittests/snapshot_tests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -395,7 +395,7 @@ BOOST_AUTO_TEST_CASE_TEMPLATE(test_compatible_versions, SNAPSHOT_SUITE, snapshot
std::string current_version = "v5";

int ordinal = 0;
for(std::string version : {"v2", "v3", "v4" /*, "v5"*/})
for(std::string version : {"v2", "v3", "v4" , "v5"})
{
if(save_snapshot && version == current_version) continue;
static_assert(chain_snapshot_header::minimum_compatible_version <= 2, "version 2 unit test is no longer needed. Please clean up data files");
Expand Down

3 comments on commit 768d61f

@matthewdarwin
Copy link

Choose a reason for hiding this comment

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

Are these snapshots compatible with the eosio 2.1 snapshots called "v5"? If not, can you bump the version number to "v6"?

@tbfleming
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is, as long as no 2.1 hardforks were activated.

@matthewdarwin
Copy link

Choose a reason for hiding this comment

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

Excellent.

Please sign in to comment.