Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mail subscription sending delay to after response #6648

Merged
merged 6 commits into from Dec 30, 2019

Conversation

alexrayne
Copy link
Contributor

@alexrayne alexrayne commented Dec 22, 2019

Reason: if some topic have many subscriptions, mailing them take much time - about 1sec/letter. For 70 subscriptions mailng accupy more then minute. Users don`t wait this time, they just go away from forum.

Changes:
here provided delay of subscription mailing after http request responded. Therefore, subscription placed in event onAfterRespond. before mail, it flushes and close http session -thus response go to browser, and after that starts to mail all the time it need.

alexrayne added 5 commits Dec 22, 2019
…p request have answered.

* subscriptions sending now starts at event onAfterRespond. So, responding to user request
  have sent as faster as can, and after that starts mailing for a looong time.

--HG--
branch : mail-dpc
…tus of subscription mailing

--HG--
branch : mail-dpc
…ena update?

*       session close lazy - when no subscriptions are mailed, do not force session close. sent http answer ordinary way

--HG--
branch : mail-dpc
@alexrayne alexrayne changed the title Mail subscription send delay after response Mail subscription sending delay to after response Dec 22, 2019
@810 810 merged commit b617f7f into Kunena:K5.1 Dec 30, 2019
3 checks passed
810 pushed a commit that referenced this issue Jun 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants