Solution The set command cache does not take effect (#3024)

Mixficsol · wuxianrong · web-flow · commit 26d10e681fd4 · 2025-02-21T18:07:27.000+08:00
Co-authored-by: wuxianrong &lt;wuxianrong@360.cn&gt;
diff --git a/src/pika_kv.cc b/src/pika_kv.cc
@@ -111,7 +111,7 @@ void SetCmd::DoUpdateCache() {
   }
   if (s_.ok()) {
     if (has_ttl_) {
-      db_->cache()->Setxx(key_, value_, ttl_millsec / 1000);
+      db_->cache()->Setxx(key_, value_, ttl_millsec > 0 ? ttl_millsec / 1000 : ttl_millsec);
     } else {
       db_->cache()->SetxxWithoutTTL(key_, value_);
     }
@@ -191,7 +191,7 @@ void GetCmd::DoUpdateCache() {
     return;
   }
   if (s_.ok()) {
-    db_->cache()->WriteKVToCache(key_, value_, ttl_millsec_ / 1000);
+    db_->cache()->WriteKVToCache(key_, value_, ttl_millsec_ > 0 ? ttl_millsec_ / 1000 : ttl_millsec_);
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,7 @@ void SetCmd::DoUpdateCache() {`
`111`	`111`	`}`
`112`	`112`	`if (s_.ok()) {`
`113`	`113`	`if (has_ttl_) {`
`114`		`- db_->cache()->Setxx(key_, value_, ttl_millsec / 1000);`
	`114`	`+ db_->cache()->Setxx(key_, value_, ttl_millsec > 0 ? ttl_millsec / 1000 : ttl_millsec);`
`115`	`115`	`} else {`
`116`	`116`	`db_->cache()->SetxxWithoutTTL(key_, value_);`
`117`	`117`	`}`
`@@ -191,7 +191,7 @@ void GetCmd::DoUpdateCache() {`
`191`	`191`	`return;`
`192`	`192`	`}`
`193`	`193`	`if (s_.ok()) {`
`194`		`- db_->cache()->WriteKVToCache(key_, value_, ttl_millsec_ / 1000);`
	`194`	`+ db_->cache()->WriteKVToCache(key_, value_, ttl_millsec_ > 0 ? ttl_millsec_ / 1000 : ttl_millsec_);`
`195`	`195`	`}`
`196`	`196`	`}`
`197`	`197`