Skip to content

Commit

Permalink
catalog: Remove deploy gen req from savepoint
Browse files Browse the repository at this point in the history
Previously, the catalog required a deploy generation when opening in
savepoint mode. However, the upgrade checker has no access to a deploy
generation and was failing to open a savepoint catalog. Ideally, the
upgrade checker would have a deploy generation so that it can test the
exact code path that an upgrade would take. For now, we're removing the
requirement to unblock the upgrade checker.
  • Loading branch information
jkosh44 authored and rjobanp committed Sep 27, 2024
1 parent c7a3516 commit e926165
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/catalog/src/durable/persist.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1240,7 +1240,7 @@ impl UnopenedPersistCatalogState {
.into());
}
(
Mode::Writable | Mode::Savepoint,
Mode::Writable,
FenceableToken::Initializing {
current_deploy_generation: None,
..
Expand Down

0 comments on commit e926165

Please sign in to comment.