After upgrading to v3.1.8 the posting has become much slower. Sometimes it can take around approximately 10 seconds for a post containing only text to be saved. With the previously installed version (2.2.2) it was almost instant.

Is there a know bug (it often seems quite slow making a post on this support forum) which causes this and a possible workaround? 

BWG 

It possible that the email sending is causing this.

can you do a test and remove the line..


from the postmessage.ascx.cs and then try to post a message to an existing topic

It possible that the email sending is causing this.

can you do a test and remove the line..

 this.Get<ISendNotification>().ToWatchingUsers(newMessage);

from the postmessage.ascx.cs and then try to post a message to an existing topic

Just tested and it doesn't seem to make much difference. Perhaps a little quicker but still very slow. (I also PM'd you with more info a little earlier)

I'm doing more testing at the moment using SQL Server Profile. I've attached a trace from my local dev server generated when posting a reply to an existing topic. 

 YAF_3_1_8ii.zip You have insufficient rights to see the content.

This is the SQL which appears to be very slow. It's taking about 6 seconds and is pulling back all 330,000 records in the database on my dev server.  I'm not sure what it is doing but could it be modified to bring back only top x rows?






Further testing shows that the message is inserted into the database very quickly. 

If I open another browser and login as a different user I can see the new message before the browser which I am posting from refreshes. So the slowdown is happening after the database insert.


Edit: I've identified the code causing the slowdown. It is in YAF.Core\Model\MessageRepositoryExtensions.cs and I comment out the lines below the performance is massively improved. However, I'm not really sure what that code does and whether it is needed?




Edit 2: It's the update to the forum stats which is causing the problems. Commenting out as below makes a massive difference to posting time.


in AF.Core\Model\MessageRepositoryExtensions.cs:
public static Message SaveNew
public static void Approve
 

Thanks for the report, issue should be fixed now in the source repo

This is the SQL which appears to be very slow. It's taking about 6 seconds and is pulling back all 330,000 records in the database on my dev server.  I'm not sure what it is doing but could it be modified to bring back only top x rows?

exec sp_executesql N'SELECT "dbo"."yaf_Message"."MessageID" AS "ID", "dbo"."yaf_Message"."TopicID", "dbo"."yaf_Message"."ReplyTo", "dbo"."yaf_Message"."Position", "dbo"."yaf_Message"."Indent", "dbo"."yaf_Message"."UserID", "dbo"."yaf_Message"."UserName", "dbo"."yaf_Message"."Posted", "dbo"."yaf_Message"."Message" AS "MessageText", "dbo"."yaf_Message"."IP", "dbo"."yaf_Message"."Edited", "dbo"."yaf_Message"."Flags", "dbo"."yaf_Message"."EditReason", "dbo"."yaf_Message"."IsModeratorChanged", "dbo"."yaf_Message"."DeleteReason", "dbo"."yaf_Message"."EditedBy", "dbo"."yaf_Message"."ExternalMessageId", "dbo"."yaf_Message"."ReferenceMessageId", "dbo"."yaf_Message"."UserDisplayName" 
FROM "dbo"."yaf_Topic" INNER JOIN "dbo"."yaf_Message" ON ("dbo"."yaf_Message"."TopicID" = "dbo"."yaf_Topic"."TopicID")
WHERE ((("dbo"."yaf_Topic"."ForumID" = @0) AND (("dbo"."yaf_Topic"."Flags" & @1) <> @2)) AND (("dbo"."yaf_Message"."Flags" & @3) = @4))
ORDER BY "dbo"."yaf_Message"."Posted" DESC',N'@0 int,@1 int,@2 int,@3 int,@4 int',@0=2,@1=8,@2=8,@3=24,@4=16

Further testing shows that the message is inserted into the database very quickly. 

If I open another browser and login as a different user I can see the new message before the browser which I am posting from refreshes. So the slowdown is happening after the database insert.


Edit: I've identified the code causing the slowdown. It is in YAF.Core\Model\MessageRepositoryExtensions.cs and I comment out the lines below the performance is massively improved. However, I'm not really sure what that code does and whether it is needed?

        if (flags.IsApproved)
        {
            repository.Approve(newMessageId, forum.ID);
            BoardContext.Current.Get<IRaiseEvent>;().Raise(new UpdateForumStatsEvent(forum.ID));
        }

Edit 2: It's the update to the forum stats which is causing the problems. Commenting out as below makes a massive difference to posting time.

BoardContext.Current.Get<IRaiseEvent>().Raise(new UpdateForumStatsEvent(forumId));

in AF.Core\Model\MessageRepositoryExtensions.cs:
public static Message SaveNew
public static void Approve
 

I just wanted to say, you freaking rock for digging into this and finding a solution!