milvus icon indicating copy to clipboard operation
milvus copied to clipboard

[Enhancement]: Use serializable read instead of linearizable read for etcd requests

Open LoveEachDay opened this issue 1 year ago • 2 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

What would you like to be added?

Milvus have several periodically read requests for etcd such as:

  1. rootcoord import manager will periodically check import tasks
  2. config will also periodically read milvus config from etcd under

And by default etcd client's Get request is linearizable, we'd provide a WithSerializable() option.

Why is this needed?

Linearizable read is relatively heavy, we can refactor our code to decrease the impact for etcd.

Anything else?

No response

LoveEachDay avatar May 06 '23 08:05 LoveEachDay

/assign @xiaofan-luan

yanliang567 avatar May 06 '23 08:05 yanliang567

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Rotten issues close after 30d of inactivity. Reopen the issue with /reopen.

stale[bot] avatar Jun 08 '23 17:06 stale[bot]