Commit 7ee2de42 authored by Kirill Smelkov's avatar Kirill Smelkov

xnet/lonet: Fix wrt recent crawshaw.io/sqlite

- sqliteutil moved to -> sqlitex (https://github.com/crawshaw/sqlite/commit/e68ccf033b)
- Poll moved from sqlite to sqlitex (https://github.com/crawshaw/sqlite/commit/1967107395)
- Poll now takes full context, not only done channel (https://github.com/crawshaw/sqlite/commit/93b88b3c5c)
parent 33f0301f
// Copyright (C) 2018 Nexedi SA and Contributors. // Copyright (C) 2018-2019 Nexedi SA and Contributors.
// Kirill Smelkov <kirr@nexedi.com> // Kirill Smelkov <kirr@nexedi.com>
// //
// This program is free software: you can Use, Study, Modify and Redistribute // This program is free software: you can Use, Study, Modify and Redistribute
// it under the terms of the GNU General Public License version 3, or (at your // it under the terms of the GNU General Public License version 3, or (at your
...@@ -26,7 +26,7 @@ import ( ...@@ -26,7 +26,7 @@ import (
"fmt" "fmt"
"crawshaw.io/sqlite" "crawshaw.io/sqlite"
"crawshaw.io/sqlite/sqliteutil" "crawshaw.io/sqlite/sqlitex"
"lab.nexedi.com/kirr/go123/xerr" "lab.nexedi.com/kirr/go123/xerr"
"lab.nexedi.com/kirr/go123/xnet/virtnet" "lab.nexedi.com/kirr/go123/xnet/virtnet"
...@@ -48,7 +48,7 @@ import ( ...@@ -48,7 +48,7 @@ import (
const schemaVer = "lonet.1" const schemaVer = "lonet.1"
type sqliteRegistry struct { type sqliteRegistry struct {
dbpool *sqlite.Pool dbpool *sqlitex.Pool
uri string // URI db was originally opened with uri string // URI db was originally opened with
} }
...@@ -60,7 +60,7 @@ func openRegistrySQLite(ctx context.Context, dburi, network string) (_ *sqliteRe ...@@ -60,7 +60,7 @@ func openRegistrySQLite(ctx context.Context, dburi, network string) (_ *sqliteRe
r := &sqliteRegistry{uri: dburi} r := &sqliteRegistry{uri: dburi}
defer r.regerr(&err, "open") defer r.regerr(&err, "open")
dbpool, err := sqlite.Open(dburi, 0, /* poolSize= */16) // XXX pool size ok? dbpool, err := sqlitex.Open(dburi, 0, /* poolSize= */16) // XXX pool size ok?
if err != nil { if err != nil {
return nil, err return nil, err
} }
...@@ -87,7 +87,7 @@ func (r *sqliteRegistry) Close() (err error) { ...@@ -87,7 +87,7 @@ func (r *sqliteRegistry) Close() (err error) {
// //
// connection is first allocated from dbpool and put back after call to f. // connection is first allocated from dbpool and put back after call to f.
func (r *sqliteRegistry) withConn(ctx context.Context, f func(*sqlite.Conn) error) error { func (r *sqliteRegistry) withConn(ctx context.Context, f func(*sqlite.Conn) error) error {
conn := r.dbpool.Get(ctx.Done()) conn := r.dbpool.Get(ctx)
if conn == nil { if conn == nil {
// either ctx cancel or dbpool close // either ctx cancel or dbpool close
if ctx.Err() != nil { if ctx.Err() != nil {
...@@ -102,13 +102,13 @@ func (r *sqliteRegistry) withConn(ctx context.Context, f func(*sqlite.Conn) erro ...@@ -102,13 +102,13 @@ func (r *sqliteRegistry) withConn(ctx context.Context, f func(*sqlite.Conn) erro
var errNoRows = errors.New("query: empty result") var errNoRows = errors.New("query: empty result")
var errManyRows = errors.New("query: multiple results") var errManyRows = errors.New("query: multiple results")
// query1 is like sqliteutil.Exec but checks that exactly 1 row is returned. // query1 is like sqlitex.Exec but checks that exactly 1 row is returned.
// //
// if query results in no rows - errNoRows is returned. // if query results in no rows - errNoRows is returned.
// if query results in more than 1 row - errManyRows is returned. // if query results in more than 1 row - errManyRows is returned.
func query1(conn *sqlite.Conn, query string, resultf func(stmt *sqlite.Stmt), argv ...interface{}) error { func query1(conn *sqlite.Conn, query string, resultf func(stmt *sqlite.Stmt), argv ...interface{}) error {
nrow := 0 nrow := 0
err := sqliteutil.Exec(conn, query, func(stmt *sqlite.Stmt) error { err := sqlitex.Exec(conn, query, func(stmt *sqlite.Stmt) error {
if nrow == 1 { if nrow == 1 {
return errManyRows return errManyRows
} }
...@@ -131,7 +131,7 @@ func (r *sqliteRegistry) setup(ctx context.Context, network string) (err error) ...@@ -131,7 +131,7 @@ func (r *sqliteRegistry) setup(ctx context.Context, network string) (err error)
return r.withConn(ctx, func(conn *sqlite.Conn) (err error) { return r.withConn(ctx, func(conn *sqlite.Conn) (err error) {
// NOTE: keep in sync wrt top-level text. // NOTE: keep in sync wrt top-level text.
err = sqliteutil.ExecScript(conn, ` err = sqlitex.ExecScript(conn, `
CREATE TABLE IF NOT EXISTS hosts ( CREATE TABLE IF NOT EXISTS hosts (
hostname TEXT NON NULL PRIMARY KEY, hostname TEXT NON NULL PRIMARY KEY,
osladdr TEXT NON NULL osladdr TEXT NON NULL
...@@ -148,7 +148,7 @@ func (r *sqliteRegistry) setup(ctx context.Context, network string) (err error) ...@@ -148,7 +148,7 @@ func (r *sqliteRegistry) setup(ctx context.Context, network string) (err error)
// do whole checks/init under transaction, so that there is // do whole checks/init under transaction, so that there is
// e.g. no race wrt another process setting config. // e.g. no race wrt another process setting config.
defer sqliteutil.Save(conn)(&err) defer sqlitex.Save(conn)(&err)
// check/init schema version // check/init schema version
ver, err := r.config(conn, "schemaver") ver, err := r.config(conn, "schemaver")
...@@ -212,7 +212,7 @@ func (r *sqliteRegistry) config(conn *sqlite.Conn, name string) (value string, e ...@@ -212,7 +212,7 @@ func (r *sqliteRegistry) config(conn *sqlite.Conn, name string) (value string, e
func (r *sqliteRegistry) setConfig(conn *sqlite.Conn, name, value string) (err error) { func (r *sqliteRegistry) setConfig(conn *sqlite.Conn, name, value string) (err error) {
defer xerr.Contextf(&err, "config: set %q = %q", name, value) defer xerr.Contextf(&err, "config: set %q = %q", name, value)
err = sqliteutil.Exec(conn, err = sqlitex.Exec(conn,
"INSERT OR REPLACE INTO meta (name, value) VALUES (?, ?)", nil, "INSERT OR REPLACE INTO meta (name, value) VALUES (?, ?)", nil,
name, value) name, value)
return err return err
...@@ -223,7 +223,7 @@ func (r *sqliteRegistry) Announce(ctx context.Context, hostname, osladdr string) ...@@ -223,7 +223,7 @@ func (r *sqliteRegistry) Announce(ctx context.Context, hostname, osladdr string)
defer r.regerr(&err, "announce", hostname, osladdr) defer r.regerr(&err, "announce", hostname, osladdr)
return r.withConn(ctx, func(conn *sqlite.Conn) error { return r.withConn(ctx, func(conn *sqlite.Conn) error {
err := sqliteutil.Exec(conn, err := sqlitex.Exec(conn,
"INSERT INTO hosts (hostname, osladdr) VALUES (?, ?)", nil, "INSERT INTO hosts (hostname, osladdr) VALUES (?, ?)", nil,
hostname, osladdr) hostname, osladdr)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment