[11045][patch] Multiple MySQL connections to database

[Ambal]

Всем трямс.

Очередной патч, который гипотетически снова должен что-то улучшить Оригинальная тема на офф форуме: Multiple MySQL connections to database

Вкратце по патчу:
1) переписан код по работе с базами данных. Также проведен рефакторинг.
2) используем несколько соединений с БД, отдельно для синхронных (SELECT) и асинхронных (INSERT+UPDATE+DELETE) запросов. Транзакции обрабатываются в отдельном потоке и только на асинхронном соединении!
3) используем меньшее количество мьютексов для работы. Класс SqlTransaction теперь также не нуждается в мьютексах.

Краткая статистика по типу SQL запросов, которые мы используем в коде мангоса:
1) SELECT - 398
2) INSERT - 88
3) DELETE - 255
4) UPDATE - 158

Чисто теоретически должно повысить производительность сервера. На практике может обернуться траблами с сохранностью данных в БД (что маловероятно) либо их релевантностью при чтении из базы (читаем про уровни изоляции транзакций).

ВНИМАНИЕ: обязательно сделайте бекап базы данных перед тестом! В случае если вдруг начнут обнаруживаться проблемы с целостностью данных - мы откажемся от идеи нескольких параллельных соединений к базе!

Репозиторий: https://github.com/Ambal/mangos
Ветка: master

Любые пожелания и комментарии приветствуются

[zhenya]

В кобольде сталкивались с проблемой, что персонаж не успевал сохраниться до попытки входа в мир при логаутах. Решение было метить транзакции отметкой содержащей гуид и в местах, где возможны траблы с чтением из базы использовались проверки нет ли незавершенной транзакции на этого персонажа.. но мы разделяли конекты (1 занимался запросами SELECT, 2 запросами INSERT+DELETE+REPLACE) насколько я помню.

[Ambal]

Сообщение от zhenya

В кобольде сталкивались с проблемой, что персонаж не успевал сохраниться до попытки входа в мир при логаутах.

В текущем ядре по большому счету сие тоже теоретически возможно - все транзакции выполняются асинхронно. Т.е. если у вас переполненная очередь асинхронных запросов и игрок быстро перелогинивается, то его данные могут неуспеть сохраниться. Я добавлю возможность выполнить транзакцию синхронно - пригодиться если нужно будет подобное проделать, скажем, при старте сервера.

P.S. В этом патче мы также разделяем коннекты: 1) синхронный для SELECT 2) асинхронный для INSERT+DELETE+UPDATE