fix CR

Author: jiacai2050

proxy/src/lib.rs

@@ -323,6 +323,7 @@ impl Proxy {
         part_info: &PartitionInfo,
     ) -> Result<TableInfo> {
         let get_inner = |i| async move {
+            // TODO: the remote engine should provide a method to get all sub table names.
             let sub_partition_table_name = util::get_sub_partition_name(base_name, part_info, i);
             let table = self
                 .instance
@@ -344,16 +345,12 @@ impl Proxy {
             Ok(table)
         };
 
-        let part_num = part_info.get_definition_num();
-        if part_num == 1 {
-            return get_inner(0).await;
-        }
-
-        // Loop get sub tables to get table info in case of some of them has problems.
-        for i in 0..part_info.get_definition_num() - 1 {
+        let part_num = part_info.get_partition_num();
+        // Loop all sub tables to get table info in case of some of them has problems.
+        for i in 0..part_num - 1 {
             let ret = get_inner(i).await;
-            if let Err(err) = ret {
-                warn!("Failed to get table info, err:{err:?}");
+            if let Err(e) = ret {
+                warn!("Failed to get table info, err:{e:?}");
             } else {
                 return ret;
             }

table_engine/src/partition/mod.rs

@@ -100,7 +100,7 @@ impl PartitionInfo {
     }
 
     #[inline]
-    pub fn get_definition_num(&self) -> usize {
+    pub fn get_partition_num(&self) -> usize {
         match self {
             Self::Random(v) => v.definitions.len(),
             Self::Hash(v) => v.definitions.len(),