From 9edfbe7718ef5826bedf0bb6d0a6054baf9b9b8e Mon Sep 17 00:00:00 2001
From: Hanefi Onaldi <Hanefi.Onaldi@microsoft.com>
Date: Wed, 5 Jan 2022 02:41:42 +0300
Subject: [PATCH] Fix the default value for DeferShardDeleteOnMove

The default for GUC citus.defer_drop_after_shard_move is true. However
we initialize the global variable with a false value.
---
 src/backend/distributed/operations/repair_shards.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/backend/distributed/operations/repair_shards.c b/src/backend/distributed/operations/repair_shards.c
index bb818552a..734d97450 100644
--- a/src/backend/distributed/operations/repair_shards.c
+++ b/src/backend/distributed/operations/repair_shards.c
@@ -112,7 +112,7 @@ PG_FUNCTION_INFO_V1(master_copy_shard_placement);
 PG_FUNCTION_INFO_V1(citus_move_shard_placement);
 PG_FUNCTION_INFO_V1(master_move_shard_placement);
 
-bool DeferShardDeleteOnMove = false;
+bool DeferShardDeleteOnMove = true;
 
 double DesiredPercentFreeAfterMove = 10;
 bool CheckAvailableSpaceBeforeMove = true;