milvus/internal/allocator
XuanYang-cn 6c5be83ea1
Refactor NewEtcdKV API (#7075)
This PR mainly did the following 3 things:

- Refactor NewEtcdKV API
- Add unittests for `kv/etcd/`, raise the coverage
up to 94%
- Correct some APIs from version to revision

The old NewEtcdKV() API took `clientv3.Client` as
one of the input, making everyone who using this API
had to create a new `clientv3.Client`. So there're a lot
of repeated codes everywhere.

Meanwhile, someplace didn't even check whether the
`clientv3.New()` returned error or not, and the old API
also didn't check whether the clientv3.Client was Nil.
This caused issue#6955.

Ectd's version and revision have nothing alike, they
represent different concepts, but our APIs kind of
mixed them all up. So I correct those APIs' names.

Resolves: #6955

Signed-off-by: yangxuan <xuan.yang@zilliz.com>
2021-08-13 11:04:09 +08:00
..
allocator.go Cancel local timestamp caching logic (#5327) 2021-05-21 05:11:21 +00:00
global_id_test.go Refactor NewEtcdKV API (#7075) 2021-08-13 11:04:09 +08:00
global_id.go fix tso (#6500) 2021-07-14 17:11:54 +08:00
id.go Change retry times (#6083) 2021-06-24 19:05:06 +08:00
OWNERS [skip ci]Update approvers (#7063) 2021-08-12 19:36:09 +08:00