Насколько безопасно сохранять модель в отдельном потоке?

Question

[Антон Иванов]

Всем привет.

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

Насколько оправдан такой подход?

Thread.new do
  model_instance.save
end

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

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

Спасибо.

Comments

[Dmitri Sinitsa]

Есть один прикол с autoloading - я сам на эти грабли напал. Если внутри треда начнётся процесс автолоадинга чего-либо, то это всё дело может (не обязательно, но может) повиснуть навсегда )) Решение: require того, что может быть нужно внутри треда. Проблема более актуальна для development окружения, потому что в проде работает eager loading, но всё равно. Довольно неочевидная вещь, на дебаггинг и гуглинг которой я потратил кучу времени.

П.С Как по мне так лучше юзать сервис фоновых задач (activejob)

Answer 1

[Виктор]

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

Но, если по конкретно по делу, то вам нужно иметь ввиду, что каждый поток будет открыт со своим коннектом к БД, который потом нужно будет закрыть руками внутри блока после сохранения:

Thread.new do
  model_instance.save
  ActiveRecord::Base.connection.close
end

Либо действия внутри потока совершать с использованием уже имеющегося соединения из пула:

Thread.new do
  ActiveRecord::Base.connection_pool.with_connection do
    model_instance.save
  end
end

Comments

[Антон Иванов]

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

Сейчас так и сделано, но не хотелось тратить серверное время на сериализацию/десериализацию модели, постановку задачи в очередь и так далее.

Но, если по конкретно по делу, то вам нужно иметь ввиду, что каждый поток будет открыт со своим коннектом к БД

Вот за это уточнение спасибо!

Answer 2

[Евгений Козлов]

Я бы начал с БД. Проанализировал план запроса, оптимизировал запрос. Выявить причину по которой БД долго выполняет запрос - первостепенно.

Comments

[Антон Иванов]

Причина, по которой БД долго выполняет запрос известна, это тоже будет правиться. Цель - убрать задержку сейчас