Как в Yii2 обработать одновременные запросы на вставку одинаковых значений в БД?

Question

[Дмитрий Ким]

Есть модуль аукциона для объявлений Items. Есть модель "Ставка" (id, user_id, item_id, price).
В таблице ставок нет уникального индекса на item_id+price.

Есть ли возможность в Yii2 предотвратить вставку одинаковых значений предложенных цен разными пользователями для определенной модели объявления item_id?
С виду кажется, что нужно просто прописать уникальный валидатор в модель ставки для валидации пары значений item_id + price, но на деле при одновременной посылке одинаковых запросов, обе проходят валидацию, а при сохранении в базу ошибок не выводится.
Варианты?

Answer 1

[Максим Тимофеев]

Не писать сразу в базу, а создать очередь, в виде промежуточной таблицы или кеша или файла. И вот из нее брать по одной записи и заносить в боевую (например кроном раз в минуту), тогда не будет одновременности.
Как вариант еще транзакции:
www.yiiframework.com/doc-2.0/yii-db-transaction.html
www.yiiframework.com/doc-2.0/guide-db-active-recor...

Comments

[Дмитрий Ким]

Не писать сразу в базу, а создать очередь, в виде промежуточной таблицы или кеша или файла. И вот из нее брать по одной записи и заносить в боевую (например кроном раз в минуту), тогда не будет одновременности.

Т.е. человек добавил предложение цены, а ему через минуту придет сообщение, что оказывается такая цена уже была предложена ранее?

[Дмитрий Ким]

Транзакции не помогли, т.к. вставка и валидация идут отдельно. Может как-то лочить таблицу в beforeValidate, и разлочивать в afterValidate?

[Максим Тимофеев]

Дмитрий Ким, Выводите "ваша ставка обрабатывается" и индикатор работы како-то, а потом "ваша ставка принята" или "упс, кто-то уже сделал такую ставку"
Но мне странно слышать:

но на деле при одновременной посылке одинаковых запросов, обе проходят валидацию, а при сохранении в базу ошибок не выводится

Если в базе стоит unique - не должно сохранятся
Надо решить вопрос с

В таблице ставок нет уникального индекса на item_id+price

. Ну и транзакции посмотрите, ссылки дал.

[Дмитрий Ким]

Максим Тимофеев, повесил уникальный индекс, на save повесил try/catch, отлавливаю IntegrityException с кодом 23000 (дупликейт ентри). По-другому "просто" не получается. При этом есть соответствующие правила валидации, но вдруг кто-от отправит одновременно.
Говнокод?

[Максим Тимофеев]

Дмитрий Ким, ну и не надо, на мой взгляд, вполне правильный вариант

[Vitaliy Orlov]

Дмитрий Ким, все правильно сделал. Другой вариант вместо try/catch делать в запросе INSERT IGNORE, это уберет overhead на проверку исключений, но зачастую это не важно

[Дмитрий Ким]

Vitaliy Orlov, все же родной parent::save(..) куда лучше написания Yii::$app->db->createCommand(...), поэтому оставляю пока try/catch

Answer 2

[Евгений Вольф]

В таблице ставок нет уникального индекса на item_id+price.

Почему?

С виду кажется, что нужно просто прописать уникальный валидатор в модель ставки для валидации пары значений item_id + price

Лично мне - так не кажется.

Есть ли возможность в Yii2 предотвратить вставку одинаковых значений

Эта возможность должна быть в БД, а не в Yii2, чисто логически.

при одновременной посылке одинаковых запросов, обе проходят валидацию

А как Вы одновременно генерируете одинаковые запросы? (просто интересно)

Варианты?

1. Добавить UNIQ-индекс и не изголяться
2. Блокировки

Answer 3

[Дмитрий Донковцев]

Наверно можно и без очередей:
1) Кладете проверку на существование записи и вставку в одну транзакцию;
2) Вначале транзакции вешаете блокировку на чтение\запись таблицы или строки;
3) Снимаете блокировку когда все сделаете.

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

Материалы:
https://habrahabr.ru/post/46542/
https://stackoverflow.com/questions/6621303/how-do...

Comments

[Boris Köln]

Дмитрий Донковцев

2) Вначале транзакции вешаете блокировку на чтение\запись таблицы или строки;

Строки пока нет, поэтому нечего лочить.
А лочить всю таблицу - очень плохой совет.
Надо использовать https://dev.mysql.com/doc/refman/5.7/en/miscellane...

[Дмитрий Донковцев]

Я дал общее описание решения, а это уже детали.

Answer 4

[Boris Köln]

В таблице ставок нет уникального индекса на item_id+price

А надо.

но на деле при одновременной посылке одинаковых запросов, обе проходят валидацию

https://ru.wikipedia.org/wiki/%D0%A3%D1%80%D0%BE%D...
и https://dev.mysql.com/doc/refman/5.7/en/miscellane...