From 39e0e1184ee2de230c6f203c8896d7c58f77ca6d Mon Sep 17 00:00:00 2001 From: WenyXu Date: Mon, 18 Sep 2023 11:09:56 +0000 Subject: [PATCH] fix: check version before alter region --- src/mito2/src/worker/handle_alter.rs | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/src/mito2/src/worker/handle_alter.rs b/src/mito2/src/worker/handle_alter.rs index 61cb343e8072..7251e4d394ba 100644 --- a/src/mito2/src/worker/handle_alter.rs +++ b/src/mito2/src/worker/handle_alter.rs @@ -47,6 +47,19 @@ impl RegionWorkerLoop { // Get the version before alter. let version = region.version(); +<<<<<<< Updated upstream + if version.metadata.schema_version >= request.schema_version { +======= + if version.metadata.schema_version > request.schema_version { + warn!( + "region schema version {} greater than request schema version {}", + version.metadata.schema_version, request.schema_version + ); +>>>>>>> Stashed changes + // Returns if it altered. + sender.send(Ok(Output::AffectedRows(0))); + return; + } // Checks whether we can alter the region directly. if !version.memtables.is_empty() { // If memtable is not empty, we can't alter it directly and need to flush