fix: Remove redundant isinstance checks for Redis protobuf parsing

abhijeet-dhumal · ntkathole · commit 5921645127fb · 2026-03-02T12:16:51.000+05:30
Signed-off-by: abhijeet-dhumal &lt;abhijeetdhumal652@gmail.com&gt;
diff --git a/sdk/python/feast/infra/online_stores/redis.py b/sdk/python/feast/infra/online_stores/redis.py
@@ -445,17 +445,13 @@ def _get_features_for_entity(
         ts_key = f"_ts:{feature_view}"
         ts_val = res_val.pop(ts_key)
         if ts_val:
-            res_ts.ParseFromString(
-                ts_val if isinstance(ts_val, bytes) else bytes(ts_val)
-            )
+            res_ts.ParseFromString(ts_val)
 
         res: Dict[str, ValueProto] = {}
         for feature_name, val_bin in res_val.items():
             val = ValueProto()
             if val_bin:
-                val.ParseFromString(
-                    val_bin if isinstance(val_bin, bytes) else bytes(val_bin)
-                )
+                val.ParseFromString(val_bin)
             res[feature_name] = val
 
         if not res:
