reduce scope

Author: whoisj

src/sagemaker_server.cc

@@ -190,7 +190,6 @@ SagemakerAPIServer::Handle(evhtp_request_t* req)
               return;
             }
           }
-          LOG_VERBOSE(1) << "SageMaker MME Custom Invoke Model Path";
 
           /* Extract targetModel to log the associated archive */
           const char* target_model =
@@ -649,15 +648,12 @@ SagemakerAPIServer::SageMakerMMEUnloadModel(
     target_model = model_name_hash;
   }
 
-  {
-    std::lock_guard<std::mutex> lock(models_list_mutex_);
-    if (sagemaker_models_list_.find(model_name_hash) ==
-        sagemaker_models_list_.end()) {
-      LOG_VERBOSE(1) << "Model " << target_model << " with model hash "
-                     << model_name_hash << " is not loaded." << std::endl;
-      evhtp_send_reply(req, EVHTP_RES_NOTFOUND); /* 404*/
-      return;
-    }
+  if (sagemaker_models_list_.find(model_name_hash) ==
+      sagemaker_models_list_.end()) {
+    LOG_VERBOSE(1) << "Model " << target_model << " with model hash "
+                   << model_name_hash << " is not loaded." << std::endl;
+    evhtp_send_reply(req, EVHTP_RES_NOTFOUND); /* 404*/
+    return;
   }
 
   LOG_INFO << "Unloading SageMaker TargetModel: " << target_model << std::endl;
@@ -728,11 +724,7 @@ SagemakerAPIServer::SageMakerMMEUnloadModel(
                  "result in SageMaker UNLOAD timeout.";
   }
 
-  std::string repo_parent_path;
-  {
-    std::lock_guard<std::mutex> lock(models_list_mutex_);
-    repo_parent_path = sagemaker_models_list_.at(model_name_hash);
-  }
+  std::string repo_parent_path = sagemaker_models_list_.at(model_name_hash);
 
   TRITONSERVER_Error* unregister_err = nullptr;
 
@@ -750,10 +742,8 @@ SagemakerAPIServer::SageMakerMMEUnloadModel(
 
   TRITONSERVER_ErrorDelete(unregister_err);
 
-  {
-    std::lock_guard<std::mutex> lock(models_list_mutex_);
-    sagemaker_models_list_.erase(model_name_hash);
-  }
+  std::lock_guard<std::mutex> lock(models_list_mutex_);
+  sagemaker_models_list_.erase(model_name_hash);
 }
 
 void