Don't print kvcache connection closed message when it was not opened. (…

…#4251) * Don't print kvcache connection closed message when it was not opened. * find hiredis when working with static build/test/install
ornladios · Jul 23, 2024 · 5b8a834 · 5b8a834
1 parent b8802f6
commit 5b8a834
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/cmake/adios2-config-common.cmake.in b/cmake/adios2-config-common.cmake.in
@@ -158,6 +158,11 @@ if(NOT @BUILD_SHARED_LIBS@)
     find_dependency(SQLite3)
   endif()
 
+  set(ADIOS2_HAVE_KVCACHE @ADIOS2_HAVE_KVCACHE@)
+  if(ADIOS2_HAVE_KVCACHE)
+    find_dependency(hiredis)
+  endif()
+
 
   adios2_add_thirdparty_target(pugixml)
   set(ADIOS2_USE_EXTERNAL_PUGIXML @ADIOS2_USE_EXTERNAL_PUGIXML@)

diff --git a/source/adios2/toolkit/kvcache/KVCacheCommon.cpp b/source/adios2/toolkit/kvcache/KVCacheCommon.cpp
@@ -13,12 +13,13 @@ namespace kvcache
 void KVCacheCommon::OpenConnection(std::string host, int port)
 {
     m_redisContext = redisConnect(host.c_str(), port);
-    if (m_redisContext == NULL || m_redisContext->err)
+    if (m_redisContext == nullptr || m_redisContext->err)
     {
         std::cout << "Error to connect to kvcache server: " << m_redisContext->errstr << std::endl;
         if (m_redisContext)
         {
             redisFree(m_redisContext);
+            m_redisContext = nullptr;
         }
     }
     else

diff --git a/source/adios2/toolkit/kvcache/KVCacheCommon.h b/source/adios2/toolkit/kvcache/KVCacheCommon.h
@@ -23,8 +23,8 @@ class KVCacheCommon
 {
 #ifdef ADIOS2_HAVE_KVCACHE
 private:
-    redisContext *m_redisContext;
-    redisReply *m_redisReply;
+    redisContext *m_redisContext = nullptr;
+    redisReply *m_redisReply = nullptr;
 
 public:
     ~KVCacheCommon() { CloseConnection(); }