Fix issues with DBChatStore.getAll (#78)

Andrzej Wójcik (Tigase) opened 8 years ago

During a test flight of an application, there were crashes related to @DBChatStore.getAll(for: with:)@.

Andrzej Wójcik (Tigase) commented 8 years ago

Artur, I'm assigning this task to you as you were experiencing this issue (crashes of an application).

They were caused by the concurrent use of a prepared statement from many threads. I was expecting that it may be problematic but as it was not causing issues (I was not aware that it was) I scheduled a task #6110 to fix this issue to version 2.1. However, as I was looking into application crashes I've found that I need to take care of this in version 2.0, so I've done task #6110 in version 2.0 to fix this issue.

I will release a new build 15 with these changes later today, so please check if this solves the issue for you.
Artur Hefczyc commented 8 years ago

2x crash in row. It happened when all my accounts (10) where still connecting and I opened MUC chat.
Andrzej Wójcik (Tigase) commented 8 years ago

%kobit There is no info about any crashes for build no. 15, so I need to wait to see if I will receive them as without crash logs I cannot do anything. Either way, it is rather not connected to crashes which happened before.
Artur Hefczyc commented 8 years ago

Indeed, however, I am sure I had quite a few yesterday. However, I cannot replicate the problem today. Will continue to test, closing the ticket now.
Login to comment

Issue Votes (0)

Watchers (0)

Reference

tigase/_clients/siskin-im#78