[BugFix]fix deduplicate shared buffer when read from remote

Signed-off-by: zombee0 <[email protected]>

Author: zombee0

be/src/io/cache_input_stream.cpp

@@ -146,11 +146,6 @@ Status CacheInputStream::_read_block_from_local(const int64_t offset, const int6
         _stats.skip_read_cache_bytes += read_size;
     }
 
-    if (res.ok() && sb) {
-        // Duplicate the block ranges to avoid saving the same data both in cache and shared buffer.
-        _deduplicate_shared_buffer(sb);
-    }
-
     return res;
 }
 
@@ -191,6 +186,11 @@ Status CacheInputStream::_read_blocks_from_remote(const int64_t offset, const in
             }
         }
 
+        if (sb) {
+            // Duplicate the block ranges to avoid saving the same data both in block_map and shared buffer.
+            _deduplicate_shared_buffer(sb);
+        }
+
         if (_enable_cache_io_adaptor) {
             int64_t delta_remote_bytes =
                     _sb_stream->shared_io_bytes() + _sb_stream->direct_io_bytes() - previous_remote_bytes;