enhance: [2.4] clean up logs of empty op key (#39395)

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