go-zero cache component may cache expired data

cache component may cache expired data

Open zhongyuanjia opened this issue 1 year ago • 5 comments

Describe the bug

From the above figure we can see that in extreme cases the cache will cache expired data

To Reproduce

go func() {
  c.Take(key, val, func() (interface{}, error) {
    val := getFromDB(ctx)
    // Perform some time-consuming operations, or GC jitter.
    time.Sleep(time.Second)
    return val
  })
}()

go func() {
  updateDB()
  c.Del(key)
}()

Expected behavior Dont cache expired data

Environments (please complete the following information):

OS: Macos
go-zero version 1.5.0

Mar 09 '23 10:03 zhongyuanjia

I talked about this scenario in my presentations. It's a rarely happened case. It's hard to solve it well without bringing the system too complicated.