Opened 5 years ago

Closed 5 years ago

#2086 closed improve feature (fixed)

Long delay after posting a reply

Reported by: shevek Owned by: shevek
Priority: major Milestone: 2.1
Component: BW Forum Keywords:
Cc:

Description

When replying to a thread the notification queue needs to be updated. The process seems to be rather inefficient so that the user has to wait pretty long before the website reloads with the new post.

Change History (11)

comment:1 Changed 5 years ago by shevek

  • Status changed from new to local_testing

Rewrote prepare_notificationqueue in Forums model to reduce the high number of queries.

Clear for local testing.

comment:2 Changed 5 years ago by shevek

  • Status changed from local_testing to to_alpha

comment:3 Changed 5 years ago by shevek

  • Status changed from to_alpha to testing

comment:4 Changed 5 years ago by crumbking

Seems to be faster for me.

Thought a link to the query diff would help to mark as fixed.

comment:5 Changed 5 years ago by shevek

This one: http://gitorious.org/bewelcome/rox/commit/6e3a0093d9253039a16fcf314b45b4e8810f111f

prepare_notification was rewritten. The old one is still available (prepare_notification_old) for comparison.

Last edited 5 years ago by shevek (previous) (diff)

comment:6 Changed 5 years ago by sitatara

  • Resolution set to fixed
  • Status changed from testing to closed

works much faster

comment:7 Changed 5 years ago by shevek

There still was a problem with the notificatiosn sent when a post was edited. This lead to two notifications with the one for the edit arriving first.

Fixed: https://gitorious.org/bewelcome/rox/commit/2cdb2ff3d6e0f314e64b9eb7718a6cab00a30ca8

comment:8 Changed 5 years ago by shevek

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:9 Changed 5 years ago by shevek

  • Status changed from reopened to to_alpha

comment:10 Changed 5 years ago by shevek

  • Status changed from to_alpha to testing

comment:11 Changed 5 years ago by shevek

  • Resolution set to fixed
  • Status changed from testing to closed

Even posting a reply to the translation announcement thread works pretty fast now. Closed as fixed.

Note: See TracTickets for help on using tickets.