Mirroristas/etcd
2
0
Fork 0
mirror of https://github.com/etcd-io/etcd.git synced 2024-09-27 06:25:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

mvcc/backend: clean up mutex, logging

Browse Source 
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
This commit is contained in:
Gyuho Lee 2018-05-23 12:09:03 -07:00 committed by Wenjia Zhang
parent e43224c3b6
commit f7367d94ff
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
mvcc/backend/batch_tx.go
View File

@ -136,15 +136,15 @@ func unsafeForEach(tx *bolt.Tx, bucket []byte, visitor func(k, v []byte) error)
// Commit commits a previous tx and begins a new writable one. // Commit commits a previous tx and begins a new writable one.
func (t *batchTx) Commit() { func (t *batchTx) Commit() {
t.Lock() t.Lock()
defer t.Unlock()
t.commit(false) t.commit(false)
t.Unlock()
} }
// CommitAndStop commits the previous tx and does not create a new one. // CommitAndStop commits the previous tx and does not create a new one.
func (t *batchTx) CommitAndStop() { func (t *batchTx) CommitAndStop() {
t.Lock() t.Lock()
defer t.Unlock()
t.commit(true) t.commit(true)
t.Unlock()
} }
func (t *batchTx) Unlock() { func (t *batchTx) Unlock() {
@ -162,9 +162,11 @@ func (t *batchTx) commit(stop bool) {
} }
start := time.Now() start := time.Now()
// gofail: var beforeCommit struct{} // gofail: var beforeCommit struct{}
err := t.tx.Commit() err := t.tx.Commit()
// gofail: var afterCommit struct{} // gofail: var afterCommit struct{}
commitDurations.Observe(time.Since(start).Seconds()) commitDurations.Observe(time.Since(start).Seconds())
atomic.AddInt64(&t.backend.commits, 1) atomic.AddInt64(&t.backend.commits, 1)
@ -209,21 +211,21 @@ func (t *batchTxBuffered) Unlock() {
func (t *batchTxBuffered) Commit() { func (t *batchTxBuffered) Commit() {
t.Lock() t.Lock()
defer t.Unlock()
t.commit(false) t.commit(false)
t.Unlock()
} }
func (t *batchTxBuffered) CommitAndStop() { func (t *batchTxBuffered) CommitAndStop() {
t.Lock() t.Lock()
defer t.Unlock()
t.commit(true) t.commit(true)
t.Unlock()
} }
func (t *batchTxBuffered) commit(stop bool) { func (t *batchTxBuffered) commit(stop bool) {
// all read txs must be closed to acquire boltdb commit rwlock // all read txs must be closed to acquire boltdb commit rwlock
t.backend.readTx.mu.Lock() t.backend.readTx.mu.Lock()
defer t.backend.readTx.mu.Unlock()
t.unsafeCommit(stop) t.unsafeCommit(stop)
t.backend.readTx.mu.Unlock()
} }
func (t *batchTxBuffered) unsafeCommit(stop bool) { func (t *batchTxBuffered) unsafeCommit(stop bool) {