mirror of https://github.com/tidwall/buntdb.git
Shrink: fix renaming on Windows
Changing the behavior to support buntdb across different operating systems as the following crash encountered on windows while renaming in shrink: ``` panic: buntdb: rename mydb.tmp mydb: Access is denied ```
This commit is contained in:
parent
e0e630f20f
commit
edbeb9e03c
15
buntdb.go
15
buntdb.go
|
@ -10,6 +10,7 @@ import (
|
||||||
"fmt"
|
"fmt"
|
||||||
"io"
|
"io"
|
||||||
"os"
|
"os"
|
||||||
|
"runtime"
|
||||||
"sort"
|
"sort"
|
||||||
"strconv"
|
"strconv"
|
||||||
"strings"
|
"strings"
|
||||||
|
@ -753,7 +754,7 @@ func (db *DB) Shrink() error {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
// Any failures below here are really bad. So just panic.
|
// Any failures below here are really bad. So just panic.
|
||||||
if err := os.Rename(tmpname, fname); err != nil {
|
if err := renameFile(tmpname, fname); err != nil {
|
||||||
panicErr(err)
|
panicErr(err)
|
||||||
}
|
}
|
||||||
db.file, err = os.OpenFile(fname, os.O_CREATE|os.O_RDWR, 0666)
|
db.file, err = os.OpenFile(fname, os.O_CREATE|os.O_RDWR, 0666)
|
||||||
|
@ -773,6 +774,18 @@ func panicErr(err error) error {
|
||||||
panic(fmt.Errorf("buntdb: %w", err))
|
panic(fmt.Errorf("buntdb: %w", err))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func renameFile(src, dest string) error {
|
||||||
|
var err error
|
||||||
|
if err = os.Rename(src, dest); err != nil {
|
||||||
|
if runtime.GOOS == "windows" {
|
||||||
|
if err = os.Remove(dest); err == nil {
|
||||||
|
err = os.Rename(src, dest)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
|
||||||
// readLoad reads from the reader and loads commands into the database.
|
// readLoad reads from the reader and loads commands into the database.
|
||||||
// modTime is the modified time of the reader, should be no greater than
|
// modTime is the modified time of the reader, should be no greater than
|
||||||
// the current time.Now().
|
// the current time.Now().
|
||||||
|
|
Loading…
Reference in New Issue