Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check all cases with BinaryDiscovery about properly balancing #380

Open
ArtDu opened this issue Apr 7, 2023 · 3 comments
Open

Check all cases with BinaryDiscovery about properly balancing #380

ArtDu opened this issue Apr 7, 2023 · 3 comments
Assignees
Labels

Comments

@ArtDu
Copy link
Contributor

ArtDu commented Apr 7, 2023

I want to test how the client behaves with BinDiscovery when routers changing(be added, be deleted)

@ArtDu ArtDu self-assigned this Apr 7, 2023
@ArtDu ArtDu added customer 1sp and removed 1sp labels Apr 10, 2023
@dkasimovskiy dkasimovskiy added the blocked Not ready to be implemented label Jun 13, 2023
@dkasimovskiy
Copy link
Contributor

Ждем релиз cartridge-java-testcontainers

@dkasimovskiy dkasimovskiy removed the blocked Not ready to be implemented label Jun 16, 2023
@ArtDu
Copy link
Contributor Author

ArtDu commented Jun 26, 2023

Проверили работу штатной балансировки клиента TNT – работает.

Но заметили 2 проблемы балансировки:

  1. Перезагрузили один роутер. Клиент перестал его нагружать.
    После перезагрузки второго, нагрузка возвращается на первый перегруженный роутер, а второй остается без нагрузки.
    Ситуация моделируется каждый раз.

image

Такое ощущение, что перепроверка здоровья ранее упавших роутеров выполняется только при падении следующего.
Кажется это поведение нужно поправить.

  1. Клиент с родной балансировкой работает где-то на 5% медленней, чем наш костыль с множеством клиентов: костыль 28,4rps, родная балансировка 27,1rps

Костыль:
image

Родная балансировка:
image

@ArtDu
Copy link
Contributor Author

ArtDu commented Jul 19, 2023

Мне кажется приоритетнее #404
Плюс надо будет обновить отдельным PR версию tarantool-testcontainers или изменить тесты

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants