U1DB

Merge lp://staging/~thisfred/u1db/c-begin-exclusive into lp://staging/u1db

  1. c-begin-exclusive
  2. Merge into trunk
Proposed by Eric Casteleijn
Status: Merged
Approved by: Eric Casteleijn
Approved revision: 428
Merged at revision: 435
Proposed branch: lp://staging/~thisfred/u1db/c-begin-exclusive
Merge into: lp://staging/u1db
Diff against target: 60 lines (+27/-6)
1 file modified
src/u1db.c (+27/-6)
To merge this branch: bzr merge lp://staging/~thisfred/u1db/c-begin-exclusive
Reviewer Review Type Date Requested Status
Samuele Pedroni Approve
dobey (community) Approve
Review via email: mp+128829@code.staging.launchpad.net

Commit message

Made database initialization happen in an exclusive transaction in C.

Description of the change

Made database initialization happen in an exclusive transaction in C.

To post a comment you must log in.
Revision history for this message
dobey (dobey) :
review: Approve
Revision history for this message
Samuele Pedroni (pedronis) wrote :

this needs to invoke a ROLLBACK in case of errors

review: Needs Fixing
lp://staging/~thisfred/u1db/c-begin-exclusive updated
427. By Eric Casteleijn

remerged trunk

428. By Eric Casteleijn

added rollback on fail

Revision history for this message
Samuele Pedroni (pedronis) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches