Как перенести create в фоновую задачу?

Question

[The Whiz]

Пытаюсь перенести create action в фон с помощью delayed job но не могу понять как нужно действовать со strong params. Сначала я попробовал все делать в контроллере:

def create
    MyController.delay.create_and_email(current_app_user)

    respond_to do |format|
      format.js {}
    end
  end

  def create_and_email(user)
    @user = AppUser.find_by(id: user.id)
    @link_share = @user.link_shares.build(link_share_params)
    @link_share.save
  end

Задача фэйлится потому что не понимает, что есть link_share_params

Тогда попробовал подключить модель:

Контроллер:

def create
    @user = current_app_user
    @user.delay.create_and_email(link_share_params)

    respond_to do |format|
      format.js {}
    end
  end

Model:

def create_and_email(params)
    link_share = self.link_shares.build(params)
    link_share.save
  end

Тоже провал: Job AppUser#create_and_email (id=15) FAILED (0 prior attempts) with ActiveModel::ForbiddenAttributesError: ActiveModel::ForbiddenAttributesError

Буду благодарен любому решению, перелопатил уже весь интернет.

Answer 1

[Viktor Vsk]

Я даже никогда не задумывался, что можно придумать ставить в очередь экшены контроллеров. Это ж даже в определение не укладывается. Фоновая задача = очередь [конкретных] задач. А экше контроллера может, в зависимости от контекста, как стать произвольной задачей (или множеством произвольных задач), так и отсутствием задач.

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

UPD. Модель есть, уже проще.
У вас в синхронном виде все работает для начала?

Comments

[The Whiz]

Да, работает. Что не так с постановкой экшена в очередь? :) Выглядит не очень, но в будущем можно убрать в service object. Но это не главное, на данный момент проблема в пробрасывании параметров.

[Viktor Vsk]

Denis G: от названия, например. Контроллер "контролирует", а не "выполняет" :)
Я, конечно, DJ непользуюсь. У меня resque\sidekiq. Смысла DJ не понимаю, но да ладно. Я сильно сомневаюсь, что это может быть его спецификой

Скорей всего, вы что-то со strong params не дописали и в синхронном виде вы проверяете не то же самое, что в асинхронном.

[Viktor Vsk]

Denis G: ну, или DJ не перегрузили, например, после какого-то изменнеия

[The Whiz]

я dj выбрал потому что это единственная отложенная задача, и ради нее нет резона тянуть сайдкик. А про синхронность я немного слукавил - все работает при условии если метод create сам по себе находится в контроллере (как при обычном CRUD), с моделью он не работает. Можете пожалуйста объяснить мне как перетянуть целиком create на примере user.posts.build(params) в модель? В контроллере получается останется только def create user.delay.build_post end

[Viktor Vsk]

Denis G: Не до конца понял, что значит "create только в экшене". Залейте куда-нибудь на pastie.org полный код контроллера (вместе со стронг-парамс куском), можно и с кодом модели на всякий случай.

Мне DJ в свое время не подошел тем, что он постоянно висит в памяти. И постоянно растет необходимая ему память. То есть, сделали вы какую-то задачу, которой надо выделить 2 гига памяти, а следующие сто - будут требовать 100 мегабайт. Но 2 гига никак не уменьшатся, пока не перегрузить весь процесс DJ.

Да и вообще, нет у него плюсов перед другими очередями. А тот факт, что для других нужно тянуть redis - это скорее плюс. Он и работает быстрее, и велика вероятность, что в будущем понадобится redis сам по себе.

[The Whiz]

Спасибо большое

Вот, создал pastie: pastie.org/9993494

Хочу create сунуть в фон, потому как он занимает значительное время