Code review comment for lp:~allenap/postgresfixture/more-locking

Revision history for this message
Raphaƫl Badin (rvb) wrote :

Looks good.

rvba -> allenap: could you tell me why you did s/locks/shares/ in the postgresfixtur branch?
allenap -> rvba: That change, from lock to shares, is because of the addition of the "critical section" lock (the lockf based one). The shares lock is better named now because it reflects that each acquired lock on it represents a share in the resource.

review: Approve

« Back to merge proposal