diff --git a/be/src/util/internal_service_recoverable_stub.cpp b/be/src/util/internal_service_recoverable_stub.cpp
index ea7cf464159b5..fe84f1e27b9ff 100644
--- a/be/src/util/internal_service_recoverable_stub.cpp
+++ b/be/src/util/internal_service_recoverable_stub.cpp
@@ -49,7 +49,7 @@ PInternalService_RecoverableStub::PInternalService_RecoverableStub(const butil::
 
 PInternalService_RecoverableStub::~PInternalService_RecoverableStub() = default;
 
-Status PInternalService_RecoverableStub::reset_channel(const std::string protocal) {
+Status PInternalService_RecoverableStub::reset_channel(const std::string& protocal) {
     std::lock_guard<std::mutex> l(_mutex);
     brpc::ChannelOptions options;
     options.connect_timeout_ms = config::rpc_connect_timeout_ms;
diff --git a/be/src/util/internal_service_recoverable_stub.h b/be/src/util/internal_service_recoverable_stub.h
index f6d9fcdf3a79b..d2f67116128a3 100644
--- a/be/src/util/internal_service_recoverable_stub.h
+++ b/be/src/util/internal_service_recoverable_stub.h
@@ -29,7 +29,7 @@ class PInternalService_RecoverableStub : public PInternalService,
     PInternalService_RecoverableStub(const butil::EndPoint& endpoint);
     ~PInternalService_RecoverableStub();
 
-    Status reset_channel(const std::string protocal = "");
+    Status reset_channel(const std::string& protocal = "");
 
 #ifdef BE_TEST
     PInternalService_Stub* stub() { return _stub.get(); }