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

bug: thread 'main' panicked at /risingwave/src/storage/hummock_sdk/src/version.rs:996:21: delta_type is not set #18650

Closed
kwannoel opened this issue Sep 24, 2024 · 1 comment
Assignees
Labels
Milestone

Comments

@kwannoel
Copy link
Contributor

kwannoel commented Sep 24, 2024

Describe the bug

Panic when upgrading a cluster from v1.9.2 to latest main (~2.0.0). There's no issue upgrading a cluster from v1.9.2 to v1.10.x or v1.10.x to v2.0.x.

That being said, I'm not sure if we upgrade v1.9.2 to v1.10.x, then from v1.10.x to v2.0.x, the same error will surface. So we should still investigate on what type of error this is.

Error message/log

thread 'main' panicked at /risingwave/src/storage/hummock_sdk/src/version.rs:996:21:
delta_type is not set
stack backtrace:
   0: rust_begin_unwind
             at /rustc/5affbb17153bc69a9d5d8d2faa4e399a014a211e/library/std/src/panicking.rs:665:5
   1: core::panicking::panic_fmt
             at /rustc/5affbb17153bc69a9d5d8d2faa4e399a014a211e/library/core/src/panicking.rs:74:14
   2: from<risingwave_hummock_sdk::sstable_info::SstableInfo>
             at ./src/storage/hummock_sdk/src/version.rs:996:21
   3: call_mut<fn(&risingwave_pb::hummock::GroupDelta) -> risingwave_hummock_sdk::version::GroupDeltaCommon<risingwave_hummock_sdk::sstable_info::SstableInfo>, (&risingwave_pb::hummock::GroupDelta)>
             at /rustc/5affbb17153bc69a9d5d8d2faa4e399a014a211e/library/core/src/ops/function.rs:166:5
   4: {closure#0}<&risingwave_pb::hummock::GroupDelta, risingwave_hummock_sdk::version::GroupDeltaCommon<risingwave_hummock_sdk::sstable_info::SstableInfo>, (), fn(&risingwave_pb::hummock::GroupDelta) -> risingwave_hummock_sdk::version::GroupDeltaCommon<risingwave_hummock_sdk::sstable_info::SstableInfo>, core::iter::traits::iterator::Iterator::for_each::call::{closure_env#0}<risingwave_hummock_sdk::version::GroupDeltaCommon<risingwave_hummock_sdk::sstable_info::SstableInfo>, alloc::vec::{impl#20}::extend_trusted::{closure_env#0}<risingwave_hummock_sdk::version::GroupDeltaCommon<risingwave_hummock_sdk::sstable_info::SstableInfo>, alloc::alloc::Global, core::iter::adapters::map::Map<core::slice::iter::Iter<risingwave_pb::hummock::GroupDelta>, fn(&risingwave_pb::hummock::GroupDelta) -> risingwave_hummock_sdk::version::GroupDeltaCommon<risingwave_hummock_sdk::sstable_info::SstableInfo>>>>>
             at /rustc/5affbb17153bc69a9d5d8d2faa4e399a014a211e/library/core/src/iter/adapters/map.rs:89:28

To Reproduce

Rerun https://buildkite.com/risingwavelabs/main-cron/builds/3315#01921f9d-f86b-4358-9078-cf24c47102ce.

Expected behavior

No panic

How did you deploy RisingWave?

No response

The version of RisingWave

No response

Additional context

https://buildkite.com/risingwavelabs/main-cron/builds/3315#01921f9d-f86b-4358-9078-cf24c47102ce

@hzxa21
Copy link
Collaborator

hzxa21 commented Sep 24, 2024

I suspect it is because we deprecate two delta types in this PR.

It will cause the panic here when the old delta containing the deprecated type is accessed by the new image:

None => panic!("delta_type is not set"),

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

No branches or pull requests

4 participants