Closed Bug 336314 Opened 15 years ago Closed 15 years ago

Make storage threadsafe

Categories

(Toolkit :: Storage, defect)

1.8 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: brettw, Assigned: brettw)

References

Details

(Keywords: fixed1.8.1)

Attachments

(1 file)

 
Attached patch PatchSplinter Review
Attachment #220672 - Flags: first-review?(darin)
Comment on attachment 220672 [details] [diff] [review]
Patch

Looks fine.  My only comment is that it might make sense to document the initialization requirements of the mozStorageService implementation of mozIStorageService separately from the interface.  Interfaces could be re-used in general, so documentation about a particular ContractID is better placed next to the declaration of the ContractID ;-)

r=darin
Attachment #220672 - Flags: first-review?(darin) → first-review+
On branch and trunk.
Status: NEW → RESOLVED
Closed: 15 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
as expected, this change breaks BeOS builds due to lack of pthreads support in BeOS and lack of bthreads support in sqlite.  Bezilla team has filed a bug with sqlite to document this lack.

Is it possible to disable "Places" until this problem is resolved?  --disable-places no longer seems to work.
Places is disabled by default, but mozStorage is a core part of the platform.  The best thing to do would be to simply implement the bthreads threading backend for sqlite.
(In reply to comment #5)
> Places is disabled by default, but mozStorage is a core part of the platform. 
> The best thing to do would be to simply implement the bthreads threading
> backend for sqlite.
> 
Because BeOS uses its own native threads (not pthreads), this is not really a workable solution.  Instead, we're going to need to implement bthreads under sqlite but this isn't quite so simple a task.

It would be great to find some other workaround in the meantime and any ideas would be appreciated.

You need to log in before you can comment on or make changes to this bug.