refine the check logic

Author: BohuTANG

src/query/storages/fuse/src/io/read/block_reader_parquet.rs

@@ -328,13 +328,13 @@ impl BlockReader {
         }
         let now = SystemTime::now();
         let mut final_result = Vec::new();
-        let mut merge_results = futures::future::try_join_all(merge_io_handlers).await?;
-        merge_results.sort_by(|a, b| a.0.cmp(&b.0));
+        let merge_results = futures::future::try_join_all(merge_io_handlers).await?;
         for index in indices.keys() {
             let column_meta = &part.columns_meta[index];
             let column_start = column_meta.offset;
             let column_end = column_start + column_meta.len;
 
+            // Find the range index and range.
             let range_idx = range_merger.get(column_start..column_end);
             let (idx, range) = match range_idx {
                 None => Err(ErrorCode::Internal(format!(
@@ -343,9 +343,19 @@ impl BlockReader {
                 ))),
                 Some((i, r)) => Ok((i, r)),
             }?;
-            let data = &merge_results[idx].1;
-            let column_data = data
-                [(column_start - range.start) as usize..(column_end - column_start) as usize]
+
+            // For loop but not move data.
+            let mut range_data = None;
+            for (i, data) in &merge_results {
+                if *i == idx {
+                    range_data = Some(data);
+                    break;
+                }
+            }
+
+            // Here has data copy.
+            let column_data = range_data.unwrap()[(column_start - range.start - 1) as usize
+                ..(column_end - column_start - 1) as usize]
                 .to_vec();
             final_result.push((*index, column_data));
         }