KV/Fix/Removed-Erraneous-DeadKey-Flag-From-FindOne

Author: Harshit12c

CLAUDE.md

@@ -84,7 +84,7 @@ The connector here only *reads* `meshCfg.secondaryRedisEnabled`; it's set in `sh
 ### Reads
 - **`findWithKVConnector`** (`Flow.hs:927`) → `findOneFromRedis` (`:1091`):
   - Fetches from **primary** (`getDataFromPKeysRedis meshCfg.kvRedis`, `:1105`).
-  - Checks **secondary ONLY** when `null primaryMatching && null primaryDeadRows && meshEnabled && secondaryRedisEnabled` (`:1119`). **A matching primary row short-circuits — secondary is never read.**
+  - Checks **secondary ONLY** when `null primaryMatching && meshEnabled && secondaryRedisEnabled` (`:1119`). **A matching primary (live) row short-circuits — secondary is never read.
   - Both miss → DB; recache only if `IS_CACHING_DB_FIND_ENABLED`.
   - **Implication:** stale primary copy is returned over a fresher secondary copy. No version compare.
 - **`findAllWithKVAndConditionalDBInternal`** (`Flow.hs:1529`): when `secondaryRedisEnabled`, reads **both** clouds in parallel (`createMultiCloudConfigs`, `callKVKVAsync`) and `matchAndDeduplicateKVRows`. Dedup is **by primary key only** — primary wins over secondary; KV (possibly stale) wins over fresh DB (`getUniqueDBRes` excludes DB rows whose PK is in *any* KV row, live **or dead**).

src/EulerHS/KVConnector/Flow.hs

@@ -1114,9 +1114,8 @@ findOneFromRedis meshCfg whereClause = do
             let primaryMatching = findAllMatching whereClause primaryLiveRows
             -- Check secondary cloud Redis if:
             -- 1. No matching rows in primary (even if raw rows exist)
-            -- 2. No dead rows (deleted data should not trigger fallback)
-            -- 3. Mesh and secondary enabled
-            if null primaryMatching && null primaryDeadRows && meshCfg.meshEnabled && meshCfg.secondaryRedisEnabled
+            -- 2. Mesh and secondary enabled
+            if null primaryMatching && meshCfg.meshEnabled && meshCfg.secondaryRedisEnabled
               then do
                 Metrics.withKVLatencyMetric "REDIS_FIND_ONE" modelName "secondaryCluster" $ do
                   secondaryRowsRes <- foldEither <$> mapM (getDataFromPKeysRedis meshCfg.kvRedisSecondary) (mkUniq keyRes)