search

shunfei / cronsun

A Distributed, Fault-Tolerant Cron-Style Job System.
Apache License 2.0
2.92k stars 456 forks source link

可能是bug #36

Closed towerjt closed 7 years ago

towerjt commented 7 years ago

node.go中的ISNodeFault ` func ISNodeFault(id string) (bool, error) { n := 0 err := mgoDB.WithC(Coll_Node, func(c *mgo.Collection) error { var e error n, e = c.Find(bson.M{"_id": id, "alived": true}).Count() return e })

return n > 0, err

} ` 如果返回true的话，实际上是节点有效，跟函数的命名是相反的吧？

是不是应该改成： return n == 0, err

miraclesu commented 7 years ago

这个函数命名和逻辑是反了，但在使用那边也是反的 下一版把这个处理一下，谢谢反馈