added some comments to mention default timeout to close descDb stream.

Signed-off-by: Rahman Abber Tahir <[email protected]>

src/libCli/Call.cpp

@@ -192,7 +192,7 @@ namespace cli
 
         //before calling the RPC, close the DescDb connection with a default timeout. We still continue with rpc call
         //but remove the cache file if the stream was not closed gracefully so it fetches the reflection data again next time.
-        grpc::Status dbDescStatus = ConnectionManager::getInstance().closeDescDbStream(serverAddress);
+        ConnectionManager::getInstance().closeDescDbStream(serverAddress);
 
         grpc::testing::CliCall call(channel, methodStr, clientMetadata, deadline);
 

src/libCli/ConnectionManager.cpp

@@ -73,7 +73,7 @@ namespace cli
             return grpc::Status( grpc::StatusCode::ABORTED, "descDbProxy has not been initialized.");
         }
 
-        //if proxy exists close the stream with a deadline.
+        //if proxy exists close the stream.
         grpc::Status status = m_connections[f_serverAddress].descDbProxy->closeDescDbStream();
 
         //delete the proxy, findChannelByAddress() protects from accessing uninitialzed DbProxy.

src/libCli/libCli/ConnectionManager.hpp

@@ -52,7 +52,7 @@ namespace cli
         /// @returns the gRpc DescriptorPool of the corresponding server address.
         std::shared_ptr<grpc::protobuf::DescriptorPool> getDescPool(std::string f_serverAddress, ArgParse::ParsedElement &f_parseTree);
 
-        /// @brief closes the DescDb stream with a default deadline.
+        /// @brief closes the DescDb stream with a default deadline of 2 seconds.
         /// @param f_serverAddress server addresss to lookup the assigned DescDbProxy.
         /// @return returns grpc::StatusCode::ABORTED status if no DescDb proxy is attached to the server address, 
         /// otherwise grpc status as a result of stream closure.