fix: Improve Redis cache implementation

- Add Redis availability check in tests
- Make ResetAllDB pure virtual in CacheBase
- Update CMake configuration for Redis support

omnn/storage/CMakeLists.txt

@@ -1,33 +1,48 @@
 option(OPENMIND_STORAGE_FOUNDATIONDB "Use Apple FoundationDB library for one of the multistorage levels" OFF)
 option(OPENMIND_STORAGE_LEVELDB "Use Google LevelDB library for one of the multistorage levels" ON)
+option(OPENMIND_STORAGE_REDIS "Use Redis for distributed caching" OFF)
 
 set(DEPENDENCIES rt)
-if(OPENMIND_STORAGE_LEVELDB OR OPENMIND_STORAGE_FOUNDATIONDB)
+if(OPENMIND_STORAGE_LEVELDB OR OPENMIND_STORAGE_FOUNDATIONDB OR OPENMIND_STORAGE_REDIS)
     option(OPENMIND_STORAGE_ALLOW_UPGRADE "Update optimized cached values during fetch if new optimizations applied" ON)
     option(OPENMIND_STORAGE_VOLATILE "Clean cache each start (for debugging purposes)" OFF)
 
-	if(OPENMIND_STORAGE_LEVELDB)
-		find_package(Threads) # workaround leveldb config bug
-		list(APPEND DEPENDENCIES google/leveldb)
-		if(NOT WIN32)
-			list(APPEND DEPENDENCIES pthread)
-		endif()
-	endif()
+    if(OPENMIND_STORAGE_LEVELDB)
+        find_package(Threads) # workaround leveldb config bug
+        list(APPEND DEPENDENCIES google/leveldb)
+        if(NOT WIN32)
+            list(APPEND DEPENDENCIES pthread)
+        endif()
+    endif()
 
-	if(OPENMIND_STORAGE_FOUNDATIONDB)
-		list(APPEND DEPENDENCIES ohhmm/FoundationDB)
-	endif()
+    if(OPENMIND_STORAGE_FOUNDATIONDB)
+        list(APPEND DEPENDENCIES ohhmm/FoundationDB)
+    endif()
 
-endif(OPENMIND_STORAGE_LEVELDB OR OPENMIND_STORAGE_FOUNDATIONDB)
+    if(OPENMIND_STORAGE_REDIS)
+        find_library(HIREDIS_LIBRARY NAMES hiredis)
+        find_path(HIREDIS_INCLUDE_DIR NAMES hiredis/hiredis.h)
+        if(HIREDIS_LIBRARY AND HIREDIS_INCLUDE_DIR)
+            include_directories(${HIREDIS_INCLUDE_DIR})
+            list(APPEND DEPENDENCIES ${HIREDIS_LIBRARY})
+        else()
+            message(FATAL_ERROR "hiredis library not found. Please install libhiredis-dev")
+        endif()
+    endif()
+
+endif(OPENMIND_STORAGE_LEVELDB OR OPENMIND_STORAGE_FOUNDATIONDB OR OPENMIND_STORAGE_REDIS)
 
 lib(${DEPENDENCIES})
 
 if(OPENMIND_STORAGE_LEVELDB)
-	target_compile_definitions(${this_target} PUBLIC OPENMIND_STORAGE_LEVELDB)
+    target_compile_definitions(${this_target} PUBLIC OPENMIND_STORAGE_LEVELDB)
 endif()
 if(OPENMIND_STORAGE_FOUNDATIONDB)
-	target_compile_definitions(${this_target} PUBLIC OPENMIND_STORAGE_FOUNDATIONDB)
+    target_compile_definitions(${this_target} PUBLIC OPENMIND_STORAGE_FOUNDATIONDB)
+endif()
+if(OPENMIND_STORAGE_REDIS)
+    target_compile_definitions(${this_target} PUBLIC OPENMIND_STORAGE_REDIS)
 endif()
 if(OPENMIND_STORAGE_ALLOW_UPGRADE)
-	target_compile_definitions(${this_target} PUBLIC ALLOW_UPGRADE OPENMIND_STORAGE_ALLOW_UPGRADE)
-endif()
+    target_compile_definitions(${this_target} PUBLIC ALLOW_UPGRADE OPENMIND_STORAGE_ALLOW_UPGRADE)
+endif()

omnn/storage/CacheBase.h

@@ -10,13 +10,13 @@ namespace omnn::rt::storage {
     namespace fs = boost::filesystem;
 
 class CacheBase {
+public:
     using path_str_t = fs::path;
 
-public:
     virtual std::string GetOne(const std::string_view& key) = 0;
     virtual bool Set(const std::string_view& key, const std::string_view& v) = 0;
     virtual bool Clear(const std::string_view& key) = 0;
-    virtual bool ResetAllDB(const path_str_t& path);
+    virtual bool ResetAllDB(const path_str_t& path) = 0;
     virtual ~CacheBase() {}
 };