refine comments

alibaba · Nov 16, 2020 · e90dc22 · e90dc22
1 parent b2654ad
commit e90dc22
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/core/hotspot/rule_manager.go b/core/hotspot/rule_manager.go
@@ -76,7 +76,7 @@ func LoadRules(rules []*Rule) (bool, error) {
 	isEqual := reflect.DeepEqual(currentRules, rules)
 	tcMux.RUnlock()
 	if isEqual {
-		logging.Info("[HotSpot] Load rules repetition, does not load")
+		logging.Info("[HotSpot] Load rules is the same with current rules, so ignore load operation.")
 		return false, nil
 	}
 

diff --git a/core/isolation/rule_manager.go b/core/isolation/rule_manager.go
@@ -6,7 +6,6 @@ import (
 
 	"github.com/alibaba/sentinel-golang/core/misc"
 	"github.com/alibaba/sentinel-golang/util"
-
 	"github.com/alibaba/sentinel-golang/logging"
 	"github.com/pkg/errors"
 )
@@ -18,12 +17,13 @@ var (
 )
 
 // LoadRules loads the given isolation rules to the rule manager, while all previous rules will be replaced.
+// the first returned value indicates whether do real load operation, if the rules is the same with previous rules, return false
 func LoadRules(rules []*Rule) (bool, error) {
 	rwMux.RLock()
 	isEqual := reflect.DeepEqual(currentRules, rules)
 	rwMux.RUnlock()
 	if isEqual {
-		logging.Info("[Isolation] Load rules repetition, does not load")
+		logging.Info("[Isolation] Load rules is the same with current rules, so ignore load operation.")
 		return false, nil
 	}