libi / dcron

轻量分布式定时任务库 a lightweight distributed job scheduler library
MIT License
422 stars 73 forks source link

add etcd_driver #20

Closed cs0604 closed 2 years ago

libi commented 2 years ago

非常感谢提交 pr ,目前发现一些建议优化点:

  1. 执行 go mod tidy, 更新下 go.mod 和 go.sum 。
  2. driver 的 SetHeartBeat 是为了让节点保活,与 etcd keepalive一致。为了保证外部调用逻辑保持一致(不调用SetHeartBeat 时,服务器过期即释放),建议将 keepalive 逻辑迁移至 SetHeartBeat内。
  3. err最终会在调用方统一输出到 log ,所以可以取消return err前的 log error,另外调试类的 log 也可以看情况取消。
  4. NewEtcdDriver 的参数建议直接使用 *clientv3.Config ,方便有更多配置需求时使用。