This is an automated email from the ASF dual-hosted git repository.

dataroaring pushed a commit to branch branch-1.1-lts
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-1.1-lts by this push:
     new cab7bab02a fix buffer pool lock (#12952)
cab7bab02a is described below

commit cab7bab02ac5a02585b41b3626d049d56ff40c9b
Author: Xinyi Zou <zouxiny...@gmail.com>
AuthorDate: Mon Sep 26 12:13:45 2022 +0800

    fix buffer pool lock (#12952)
---
 be/src/runtime/bufferpool/buffer_allocator.cc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/be/src/runtime/bufferpool/buffer_allocator.cc 
b/be/src/runtime/bufferpool/buffer_allocator.cc
index 55a4de1076..ed1d156df5 100644
--- a/be/src/runtime/bufferpool/buffer_allocator.cc
+++ b/be/src/runtime/bufferpool/buffer_allocator.cc
@@ -710,7 +710,7 @@ int64_t BufferPool::FreeBufferArena::GetNumCleanPages() {
 }
 
 std::string BufferPool::FreeBufferArena::DebugString() {
-    std::lock_guard<SpinLock> al(lock_);
+    // std::lock_guard<SpinLock> al(lock_);
     std::stringstream ss;
     ss << "<FreeBufferArena> " << this << "\n";
     for (int i = 0; i < NumBufferSizes(); ++i) {
@@ -720,8 +720,8 @@ std::string BufferPool::FreeBufferArena::DebugString() {
            << " free buffers: " << lists.num_free_buffers.load()
            << " low water mark: " << lists.low_water_mark
            << " clean pages: " << lists.num_clean_pages.load() << " ";
-        lists.clean_pages.iterate(
-                std::bind<bool>(Page::DebugStringCallback, &ss, 
std::placeholders::_1));
+        // lists.clean_pages.iterate(
+        //         std::bind<bool>(Page::DebugStringCallback, &ss, 
std::placeholders::_1));
         ss << "\n";
     }
     return ss.str();


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to