From 8c63e3ee1c7b539bef0da1ace74b1f3a57e8673e Mon Sep 17 00:00:00 2001
From: st1page <1245835950@qq.com>
Date: Mon, 30 Oct 2023 14:12:04 +0800
Subject: [PATCH] remove without tar size

---
 src/common/src/util/column_index_mapping.rs | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/src/common/src/util/column_index_mapping.rs b/src/common/src/util/column_index_mapping.rs
index 2b1ef2ffa66c8..2c12dc47efb11 100644
--- a/src/common/src/util/column_index_mapping.rs
+++ b/src/common/src/util/column_index_mapping.rs
@@ -41,16 +41,6 @@ impl ColIndexMapping {
         Self { target_size, map }
     }
 
-    /// Create a partial mapping which maps the subscripts range `(0..map.len())` to the
-    /// corresponding element. **This method is not recommended**, please use `with_target_size` instead, see <https://github.com/risingwavelabs/risingwave/issues/7234> for more information**
-    pub fn without_target_size(map: Vec<Option<usize>>) -> Self {
-        let target_size = match map.iter().filter_map(|x| *x).max_by_key(|x| *x) {
-            Some(target_max) => target_max + 1,
-            None => 0,
-        };
-        Self { target_size, map }
-    }
-
     pub fn into_parts(self) -> (Vec<Option<usize>>, usize) {
         (self.map, self.target_size)
     }