From a4ab9c284f50ebdd934f8b66629fc2568d257e51 Mon Sep 17 00:00:00 2001 From: qinyuguang Date: Thu, 10 Mar 2022 14:50:06 +0800 Subject: [PATCH] gdb returns result when cache set failed --- database/gdb/gdb_model_select.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/database/gdb/gdb_model_select.go b/database/gdb/gdb_model_select.go index 2fcc84b2d..308ba3693 100644 --- a/database/gdb/gdb_model_select.go +++ b/database/gdb/gdb_model_select.go @@ -538,16 +538,16 @@ func (m *Model) doGetAllBySql(sql string, args ...interface{}) (result Result, e // Cache the result. if cacheKey != "" && err == nil { if m.cacheOption.Duration < 0 { - if _, err = cacheObj.Remove(ctx, cacheKey); err != nil { - intlog.Errorf(m.GetCtx(), `%+v`, err) + if _, errCache := cacheObj.Remove(ctx, cacheKey); errCache != nil { + intlog.Errorf(m.GetCtx(), `%+v`, errCache) } } else { // In case of Cache Penetration. if result.IsEmpty() && m.cacheOption.Force { result = Result{} } - if err = cacheObj.Set(ctx, cacheKey, result, m.cacheOption.Duration); err != nil { - intlog.Errorf(m.GetCtx(), `%+v`, err) + if errCache := cacheObj.Set(ctx, cacheKey, result, m.cacheOption.Duration); errCache != nil { + intlog.Errorf(m.GetCtx(), `%+v`, errCache) } } }