From 1b1db0081f66068d1bccf3d7963cb872369468f6 Mon Sep 17 00:00:00 2001
From: Michael Stetsyuk <michael.stetsyuk@clickhouse.com>
Date: Tue, 10 Sep 2024 22:39:35 +0000
Subject: [PATCH] do not fix metadata_version if replica is read_only

---
 src/Storages/MergeTree/ReplicatedMergeTreeAttachThread.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/Storages/MergeTree/ReplicatedMergeTreeAttachThread.cpp b/src/Storages/MergeTree/ReplicatedMergeTreeAttachThread.cpp
index f28b8f9e9a8..67570d78366 100644
--- a/src/Storages/MergeTree/ReplicatedMergeTreeAttachThread.cpp
+++ b/src/Storages/MergeTree/ReplicatedMergeTreeAttachThread.cpp
@@ -122,6 +122,7 @@ Int32 ReplicatedMergeTreeAttachThread::fixReplicaMetadataVersionIfNeeded(zkutil:
 {
     const String & zookeeper_path = storage.zookeeper_path;
     const String & replica_path = storage.replica_path;
+    const bool replica_readonly = storage.is_readonly;
 
     for (size_t i = 0; i != 2; ++i)
     {
@@ -132,7 +133,7 @@ Int32 ReplicatedMergeTreeAttachThread::fixReplicaMetadataVersionIfNeeded(zkutil:
 
         const Int32 metadata_version = parse<Int32>(replica_metadata_version_str);
 
-        if (metadata_version != 0)
+        if (metadata_version != 0 || replica_readonly)
         {
             /// No need to fix anything
             return metadata_version;