Как реализовать отправку email за определенный срок до события?

Question

[Rukis]

Сайт на Laravel. Использую очереди для отправки email. Драйвер очередей Beanstalkd.

Задача: пользователь создает событие на произвольную дату и может указать, что ему требуется оповещение за n времени до наступления события.

Здесь проблем нет, вычисляем задержку и ставим задачу в очередь.
Проблема возникает при редактировании пользователем события или срока оповещения. Необходимо отредактировать задачу в очереди или создать новую.

Подскажите способ как это можно сделать наиболее корректно?

Answer 1

[vism]

Потому, что очереди немножко не для этого сделаны.
Вам надо крон задачку.

Очереди нужны для того, чтоб тяжелые операции в фон выносить.
Типо генерации отчета, отправки имейла сразу и т.п. Чтоб не задерживать скрипт.

Если очереди хотите, можно извратиться.
Сделать крон задачу, которая будет раз в 10 минут проверять события и добавлять имейлы в очередь на ближайшее время :)

Comments

[Rukis]

Почему же не для этого? Специально же есть возможность размещения отложенного задания в очереди. Или есть какие то другие варианты кроме очереди? У меня множество пользователей с множеством задач, получается крон будет каждые 10 минут переться в БД и делать выборку по всем событиям всех пользователей... А если нужна точность не 10 минут, а минута или даже несколько секунд... То есть происходит перенос очереди в БД, но в чем профит? Только нагрузка лишняя...

В случае встроенного функционала очередей и Beanstalkd получается ровно 1 запрос ровно в назначенное время. Я уже реализовал при этом требуемый функционал (удаление задачи и создание новой, при переносе события), но мне показалось что решение не очень красивое и должно существовать получше. Храню в бд ид задачи из очереди, при изменении времени удаляю старую задачу, создаю новую.

[vism]

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

[Rukis]

vism,
Несколько лет, а вы? Понятное дело, что очереди используют обычно не так, и сам использую их для фонового выполнения объемных задач. Но почему это значит, что нельзя использовать очередь для отложенного выполнения задачи, при том, что есть прямой функционал позволяющий это делать? И у меня возникает вопрос, разве крон не будет говнокодом? Вы же сами сказали что это извращение. В чем необходимость то долбить БД каждые пару минут?

Крон уж точно не для этого придуман, не для периодического дерганья системы в надежде поймать нужное время для выполнения задачи... это же костыль просто, который в некоторых случаях вообще не применим. Что предложите, если запрос к БД слишком ресурсоемкий, а выполнять его придется каждую секунду, чтобы вовремя отловить время?

Использовать крон по назначению - это добавлять в него конкретную задачу, а не вызов чего то, что будет проверять "а не пора ли?". Но так как нельзя добавлять задачи пользователей прямо в список крон, надо применить такой инструмент, с которым так работать можно. Например Beanstalkd.

Чем плох подход то?

[JhaoDa]

Rukis, драйвер Beanstalk'а позволяет удалять задание по идентификатору. Запоминайте где-то его id при создании и потом удаляйте.

[Rukis]

JhaoDa, Да, так и делаю. Интересует нет ли каких то других способов.

[vism]

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

А щас вы пытаетесь костылить.
Как не дергать каждую секунду описал в первом сообщении, но вы даже не читаете.

[vism]

Rukis, В целом совершенно даже не смутило, что нет такой функциональности через компонент и приходится костылить через Beanstalkd? )))

[Rukis]

vism, вы мне будете говорить где работать? Разве я просил? Вопрос был в другом, не отклоняйтесь от темы.

Если вы определили, что я не умею строить архитектурные решения, то легко определите и предложите верное. А заодно сможете объяснить чем плохо использовать очередь как предложил я. Или не сможете?

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

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

Если не запускать задачу по крону часто, а как вы предложили "раз в 10 минут", то как быть в ситуациях, когда крон поставил задачу в очередь, а через несколько минут событие было перенесено пользователем на другое время? В этом была изначальная проблема, если вы конечно читали мой вопрос.

[Rukis]

В целом совершенно даже не смутило, что нет такой функциональности через компонент и приходится костылить через Beanstalkd?

vism, Beanstalkd - это драйвер очереди и Laravel его поддерживает. В чем вопрос?

[vism]

Rukis,
По архитектуре:
У вас есть компонент, отправка напоминания за какое-то время до события.
События, как я понял, у вас имеют разные типы и находятся по разным таблицам, но по сути есть общий интерфейс отправки сообщений.
У вас заведена таблица в БД, которая обобщает все ваши события?

[Rukis]

vism, не знаю с чего вы это решили, ни о чем подобном я не говорил. Все события в одной таблице.

[vism]

Rukis,

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

Все события в одной таблице.

Ну тогда в чем проблема?
1 раз в 10 минут заходите в БД и делаете выборку, где время отправки события меньше текущего и сообщение не было добавлено в очередь(новое) и добавляете в очередь.
Уже в очереди при отправке дергаете данные события и юзера, проверяете что все в силе и отправляете.

То есть крон единственное что делает - проверяет наличие Новых событий и добавляет их в очередь.
Таск в очереди сам разруливает проверку актуальности и сообщение при неудавшейся отправке и т.п.

Почему плохо делать как у вас.
Вы жестко завязываете работу приложения на другую софтину.
Если вы делаете для себя, то пофиг.
Если вы делаете заказ клиенту - вы поступаете подло.
Клиент не должен волноваться о том как работает ваше приложение.
Код и приложение должны быть так сделаны. Что если клиент завтра заплатит вам бабло и закроет контракт - другой разработчик должен без проблем разобраться в нем и начать работу.
Если клиент перенесет приложение на другой сервер и Beanstalkd вдруг будет багаться, нужно иметь возможность НЕ зависеть от сторонней софтины.

Я думаю вы слышали что код должен быть слабосвязанным. А вы делаете сильную связь на уровне ПО сервера. Этот пипец как ужасно, учитывая что можно сделать все через крон.

[Rukis]

vism,

1 раз в 10 минут заходите в БД и делаете выборку, где время отправки события меньше текущего и сообщение не было добавлено в очередь(новое) и добавляете в очередь

Крон сработал, задача попала в очередь, пользователь перенес событие на 5 минут (назад или вперед), как предложите разрулить эту ситуацию? Если задача должна сама решать будет она выполнять свою работу или нет, то для чего вообще крон? Поставили задачу сразу на определенное время, время пришло, задача сама всё проверила, отменилась или выполнила работу. С какой целью уже фактически отмененная задача должна висеть в очереди, срабатывать и что то проверять, хотя ее можно просто удалить из очереди?

Вы жестко завязываете работу приложения на другую софтину.

О, поделитесь, каким драйвером очередей вы порекомендуете воспользоваться, но чтоб он работал не через "другую софтину" или внешний сервис?

[vism]

Я описал все, а вы зависли на уровне тяп ляп и в продакшн.
Если вы не хотите делать лучше и думать, то мне тут уговаривать вас нет смысла.
Я описал рабочий вариант архитектуры компонента. Гибкий, не связанный, разбитый на логические составляющие.

Копайтесь в своем болоте дальше. Типичный русский фриланс как на ладони.

[Rukis]

vism, ну я же вам нормальные вопросы задал, ничего своего вам не навязываю и если вы вдруг решили что меня надо уговаривать, то прошу вас, нет и нет. Просто, если знаете ответы, ответьте, не знаете, так и скажите, что вы сразу обиженного строите то. Код не видели, решили что он тяп ляп... К чему эти фантазии...

Я вот не знаю как решить ситуации возникающие при вашем гибком решении и не знаю какой вы используете драйвер очередей, чтобы не было зависимости от других программ или сервисов. Но вы же знаете что предлагаете? Возникает ощущение что нет, иначе бы сказали, да и дело с концом. Может вам стоит получше подумать над проблемой и решениями? Очень хотелось бы услышать нормальное обоснование, а не доводы уровня "я д'Артаньян".