search

openGemini / opengemini-client-go

CNCF openGemini Go Client Library
https://opengemini.org/
Apache License 2.0
24 stars 17 forks source link

Server rotation #131

Open wolfbolin opened 3 months ago

wolfbolin commented 3 months ago

https://github.com/openGemini/opengemini-client-go/blob/e15a322d7b9b723bcd406921aaae89964c144829/opengemini/servers_check.go#L48-L58

This may not be a reasonable way to achieve rotation. Even in Golang uint32() avoids negative numbers, this way is not good for understanding and long term operation.

shoothzj commented 2 months ago

I believe rotation works well, but it is not coordinated well with the retry mechanism. We need a better strategy. cc @hiiiik @xuthus5