mirror of
https://bitbucket.org/s_l_teichmann/mtsatellite
synced 2024-12-23 16:50:18 +01:00
Removed locking from LevelDB entirely.
It is not needed because LevelDB itself handles concurreny issues and it causes a lot of contentions and massive lags.
This commit is contained in:
parent
aad612d097
commit
e4ad3a84d8
@ -6,7 +6,6 @@ package main
|
||||
|
||||
import (
|
||||
"log"
|
||||
"sync"
|
||||
|
||||
"bitbucket.org/s_l_teichmann/mtsatellite/common"
|
||||
|
||||
@ -22,10 +21,6 @@ type LevelDBBackend struct {
|
||||
decoder common.KeyTranscoder
|
||||
|
||||
changeTracker *changeTracker
|
||||
mutex sync.RWMutex
|
||||
|
||||
priority *sync.Cond
|
||||
priorityCount int
|
||||
}
|
||||
|
||||
type LevelDBSession struct {
|
||||
@ -76,7 +71,7 @@ func NewLeveDBBackend(
|
||||
encoder: encoder,
|
||||
decoder: decoder,
|
||||
changeTracker: changeTracker,
|
||||
priority: sync.NewCond(new(sync.Mutex))}
|
||||
}
|
||||
|
||||
if !interleaved {
|
||||
if err = ldb.buildCoverage(); err != nil {
|
||||
@ -88,28 +83,6 @@ func NewLeveDBBackend(
|
||||
return
|
||||
}
|
||||
|
||||
func (ldb *LevelDBBackend) suspend() {
|
||||
ldb.priority.L.Lock()
|
||||
for ldb.priorityCount > 0 {
|
||||
ldb.priority.Wait()
|
||||
}
|
||||
ldb.priority.L.Unlock()
|
||||
}
|
||||
|
||||
func (ldb *LevelDBBackend) grab() {
|
||||
ldb.priority.L.Lock()
|
||||
ldb.priorityCount++
|
||||
ldb.priority.L.Unlock()
|
||||
}
|
||||
|
||||
func (ldb *LevelDBBackend) ungrab() {
|
||||
ldb.priority.L.Lock()
|
||||
if ldb.priorityCount--; ldb.priorityCount <= 0 {
|
||||
ldb.priority.Broadcast()
|
||||
}
|
||||
ldb.priority.L.Unlock()
|
||||
}
|
||||
|
||||
func (ldb *LevelDBBackend) buildCoverage() error {
|
||||
log.Println("INFO: Start building coverage index (this may take some time)...")
|
||||
|
||||
@ -135,18 +108,6 @@ func (ldb *LevelDBBackend) buildCoverage() error {
|
||||
return nil
|
||||
}
|
||||
|
||||
func (ldb *LevelDBBackend) doRead(f func(db *leveldb.DB)) {
|
||||
ldb.mutex.RLock()
|
||||
f(ldb.db)
|
||||
ldb.mutex.RUnlock()
|
||||
}
|
||||
|
||||
func (ldb *LevelDBBackend) doWrite(f func(db *leveldb.DB)) {
|
||||
ldb.mutex.Lock()
|
||||
f(ldb.db)
|
||||
ldb.mutex.Unlock()
|
||||
}
|
||||
|
||||
func (ldb *LevelDBBackend) NewSession() (Session, error) {
|
||||
return &LevelDBSession{ldb, nil}, nil
|
||||
}
|
||||
@ -170,7 +131,6 @@ func (ldbs *LevelDBSession) Del(hash, key []byte) (success bool, err error) {
|
||||
if key, err = ldbs.backend.decoder(key); err != nil {
|
||||
return
|
||||
}
|
||||
ldbs.backend.doWrite(func(db *leveldb.DB) {
|
||||
ro := leveldb.NewReadOptions()
|
||||
defer ro.Close()
|
||||
var data []byte
|
||||
@ -186,7 +146,6 @@ func (ldbs *LevelDBSession) Del(hash, key []byte) (success bool, err error) {
|
||||
wo := leveldb.NewWriteOptions()
|
||||
defer wo.Close()
|
||||
err = ldbs.backend.db.Delete(wo, key)
|
||||
})
|
||||
return
|
||||
}
|
||||
|
||||
@ -194,10 +153,6 @@ func (ldbs *LevelDBSession) Fetch(hash, key []byte) (value []byte, err error) {
|
||||
if key, err = ldbs.backend.decoder(key); err != nil {
|
||||
return
|
||||
}
|
||||
ldbs.backend.doRead(func(db *leveldb.DB) {
|
||||
ldbs.backend.grab()
|
||||
defer ldbs.backend.ungrab()
|
||||
|
||||
ro := leveldb.NewReadOptions()
|
||||
value, err = ldbs.backend.db.Get(ro, key)
|
||||
//if err != nil {
|
||||
@ -206,7 +161,6 @@ func (ldbs *LevelDBSession) Fetch(hash, key []byte) (value []byte, err error) {
|
||||
// log.Printf("Fetch key = '%s' len(value) = %d\n", key, len(value))
|
||||
//}
|
||||
ro.Close()
|
||||
})
|
||||
return
|
||||
}
|
||||
|
||||
@ -230,10 +184,7 @@ func (ldbs *LevelDBSession) Store(hash, key, value []byte) (exists bool, err err
|
||||
if key, err = ldbs.backend.decoder(key); err != nil {
|
||||
return
|
||||
}
|
||||
ldbs.backend.doWrite(func(db *leveldb.DB) {
|
||||
ldbs.backend.grab()
|
||||
defer ldbs.backend.ungrab()
|
||||
if exists, err = keyExists(db, key); err != nil {
|
||||
if exists, err = keyExists(ldbs.backend.db, key); err != nil {
|
||||
return
|
||||
}
|
||||
if ldbs.tx != nil {
|
||||
@ -241,9 +192,8 @@ func (ldbs *LevelDBSession) Store(hash, key, value []byte) (exists bool, err err
|
||||
return
|
||||
}
|
||||
wo := leveldb.NewWriteOptions()
|
||||
err = db.Put(wo, key, value)
|
||||
err = ldbs.backend.db.Put(wo, key, value)
|
||||
wo.Close()
|
||||
})
|
||||
if err != nil {
|
||||
return
|
||||
}
|
||||
@ -276,13 +226,11 @@ func (ldbs *LevelDBSession) CommitTransaction() (err error) {
|
||||
return
|
||||
}
|
||||
ldbs.tx = nil
|
||||
ldbs.backend.doWrite(func(db *leveldb.DB) {
|
||||
wo := leveldb.NewWriteOptions()
|
||||
wo.SetSync(true)
|
||||
err = db.Write(wo, tx)
|
||||
err = ldbs.backend.db.Write(wo, tx)
|
||||
wo.Close()
|
||||
tx.Close()
|
||||
})
|
||||
return
|
||||
}
|
||||
|
||||
@ -290,8 +238,6 @@ func (ldbs *LevelDBSession) AllKeys(
|
||||
hash []byte,
|
||||
done <-chan struct{}) (<-chan []byte, int, error) {
|
||||
|
||||
ldbs.backend.mutex.RLock()
|
||||
|
||||
ro := leveldb.NewReadOptions()
|
||||
ro.SetFillCache(false)
|
||||
|
||||
@ -305,14 +251,12 @@ func (ldbs *LevelDBSession) AllKeys(
|
||||
if err := it.GetError(); err != nil {
|
||||
it.Close()
|
||||
ro.Close()
|
||||
ldbs.backend.mutex.RUnlock()
|
||||
return nil, n, err
|
||||
}
|
||||
|
||||
keys := make(chan []byte)
|
||||
|
||||
go func() {
|
||||
ldbs.backend.mutex.RUnlock()
|
||||
defer ro.Close()
|
||||
defer close(keys)
|
||||
defer it.Close()
|
||||
@ -371,8 +315,6 @@ func (ldbs *LevelDBSession) plainSpatialQuery(
|
||||
|
||||
go func() {
|
||||
defer close(blocks)
|
||||
ldbs.backend.mutex.RLock()
|
||||
defer ldbs.backend.mutex.RUnlock()
|
||||
|
||||
ro := leveldb.NewReadOptions()
|
||||
defer ro.Close()
|
||||
@ -433,8 +375,6 @@ func (ldbs *LevelDBSession) interleavedSpatialQuery(
|
||||
|
||||
go func() {
|
||||
defer close(blocks)
|
||||
ldbs.backend.mutex.RLock()
|
||||
defer ldbs.backend.mutex.RUnlock()
|
||||
|
||||
ro := leveldb.NewReadOptions()
|
||||
defer ro.Close()
|
||||
@ -455,7 +395,6 @@ func (ldbs *LevelDBSession) interleavedSpatialQuery(
|
||||
//log.Printf("seeking to: %d\n", zmin)
|
||||
it.Seek(common.ToBigEndian(zmin))
|
||||
for it.Valid() {
|
||||
ldbs.backend.suspend()
|
||||
zcode := common.FromBigEndian(it.Key())
|
||||
|
||||
if zcode > zmax {
|
||||
|
Loading…
Reference in New Issue
Block a user