Edgewall Software

Opened 12 years ago

Last modified 22 months ago

#3444 reopened defect

Strange `database is locked` error: commit fails but data is nevertheless saved...

Reported by: Christian Boos Owned by: Jonas Borgström
Priority: normal Milestone: not applicable
Component: database backend Version: devel
Severity: normal Keywords: database lock pysqlite weird
Cc: trac@…
Release Notes:
API Changes:


I just tried to close #3410, and I got the following error message:

Traceback (most recent call last):
  File "/usr/lib/python2.3/site-packages/trac/web/main.py", line 314, in dispatch_request
  File "/usr/lib/python2.3/site-packages/trac/web/main.py", line 199, in dispatch
    resp = chosen_handler.process_request(req)
  File "/usr/lib/python2.3/site-packages/trac/ticket/web_ui.py", line 260, in process_request
    self._do_save(req, db, ticket)
  File "/usr/lib/python2.3/site-packages/trac/ticket/web_ui.py", line 530, in _do_save
OperationalError: database is locked

So far so good (well…) but when looking at the timeline immediately after that, the change appears to have succeeded! And that's really puzzling: if the commit fails, the data shouldn't persist.

This is not the first time I see somthing like this, so now I've decided to create a ticket about it. No milestone set, it's just a place to record the issue and discuss it until what happens is understood…

Attachments (0)

Change History (8)

comment:1 Changed 12 years ago by Christian Boos

Version: 0.9.6devel

comment:2 Changed 12 years ago by anonymous

Cc: trac@… added

Interested in this ticket as well.

comment:3 Changed 12 years ago by sid

The database is locked error was addressed in #3503. Can you upgrade to the version specified in that ticket and see if this issue still exists for you?

comment:4 Changed 12 years ago by Christian Boos

Resolution: wontfix
Status: newclosed

The reporter obviously knows about #3503, as he fixed that issue… so I guess the upgrade advice was for trac@… ;)

I never could reproduce that bug, I only saw it here on t.e.o when it was still using SQLite.

This issue was about a "database is locked" raised by a commit, and seeing that this commit apparently succeeded despite of this exception. I also never got replies on the pysqlite mailing list about this problem, so I think we end up with a wontfix here.

comment:5 Changed 12 years ago by trac@…

using pysqlite3 gets rid of that issue.

comment:6 Changed 12 years ago by Christian Boos

Keywords: weird added
Milestone: none
Priority: lownormal
Resolution: wontfix
Status: closedreopened

Apparently, this still happens: http://pacopablo.com/irclogs/2007/03/05#T13:53:11 (more details needed)

This could be explained if the connection would be in autocommit mode (isolation_level==None), which should normally not be the case (isolation_mode is '' by default, which means use DEFERRED transactions).

comment:7 Changed 12 years ago by anonymous

Under DEFERRED transaction, If you direct write to the database with multithread, "database is locked" will jump out, Uf you do and read action and required shared lock, this will be ok, I don't know is this a correct reaction for sqlite :<

comment:8 Changed 9 years ago by Christian Boos

Component: generaldatabase backend

Modify Ticket

Change Properties
Set your email in Preferences
as reopened The owner will remain Jonas Borgström.
The ticket will be disowned.
as The resolution will be set.
The owner will be changed from Jonas Borgström to anonymous.

Add Comment

E-mail address and name can be saved in the Preferences .
Note: See TracTickets for help on using tickets.