enhance: [2.5] clean up logs of empty op key (#39394)

cherry-pick from master: #39335 related: #39334 Signed-off-by: shaoting-huang <[email protected]>
milvus-io · Jan 20, 2025 · 8c35064 · 8c35064
1 parent 4464966
commit 8c35064
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/internal/rootcoord/rbac_task.go b/internal/rootcoord/rbac_task.go
@@ -209,12 +209,14 @@ func executeOperatePrivilegeTaskSteps(ctx context.Context, core *Core, in *milvu
 				})
 			})
 		}
-		if err := core.proxyClientManager.RefreshPolicyInfoCache(ctx, &proxypb.RefreshPolicyInfoCacheRequest{
-			OpType: opType,
-			OpKey:  funcutil.PolicyForPrivileges(expandGrants),
-		}); err != nil {
-			log.Ctx(ctx).Warn("fail to refresh policy info cache", zap.Any("in", in), zap.Error(err))
-			return nil, err
+		if len(expandGrants) > 0 {
+			if err := core.proxyClientManager.RefreshPolicyInfoCache(ctx, &proxypb.RefreshPolicyInfoCacheRequest{
+				OpType: opType,
+				OpKey:  funcutil.PolicyForPrivileges(expandGrants),
+			}); err != nil {
+				log.Ctx(ctx).Warn("fail to refresh policy info cache", zap.Any("in", in), zap.Error(err))
+				return nil, err
+			}
 		}
 		return nil, nil
 	}))