diff options
| author | 2021-08-29 15:41:41 +0100 | |
|---|---|---|
| committer | 2021-08-29 16:41:41 +0200 | |
| commit | ed462245730bd7832019bd43e0bc1c9d1c055e8e (patch) | |
| tree | 1caad78ea6aabf5ea93c93a8ade97176b4889500 /internal/db/bundb/conn.go | |
| parent | Mention fixup (#167) (diff) | |
| download | gotosocial-ed462245730bd7832019bd43e0bc1c9d1c055e8e.tar.xz | |
Add SQLite support, fix un-thread-safe DB caches, small performance f… (#172)
* Add SQLite support, fix un-thread-safe DB caches, small performance fixes
Signed-off-by: kim (grufwub) <grufwub@gmail.com>
* add SQLite licenses to README
Signed-off-by: kim (grufwub) <grufwub@gmail.com>
* appease the linter, and fix my dumbass-ery
Signed-off-by: kim (grufwub) <grufwub@gmail.com>
* make requested changes
Signed-off-by: kim (grufwub) <grufwub@gmail.com>
* add back comment
Signed-off-by: kim (grufwub) <grufwub@gmail.com>
Diffstat (limited to 'internal/db/bundb/conn.go')
| -rw-r--r-- | internal/db/bundb/conn.go | 72 | 
1 files changed, 72 insertions, 0 deletions
| diff --git a/internal/db/bundb/conn.go b/internal/db/bundb/conn.go new file mode 100644 index 000000000..698adff3d --- /dev/null +++ b/internal/db/bundb/conn.go @@ -0,0 +1,72 @@ +package bundb + +import ( +	"context" +	"database/sql" + +	"github.com/sirupsen/logrus" +	"github.com/superseriousbusiness/gotosocial/internal/db" +	"github.com/uptrace/bun" +	"github.com/uptrace/bun/dialect" +) + +// dbConn wrapps a bun.DB conn to provide SQL-type specific additional functionality +type DBConn struct { +	errProc func(error) db.Error // errProc is the SQL-type specific error processor +	log     *logrus.Logger       // log is the logger passed with this DBConn +	*bun.DB                      // DB is the underlying bun.DB connection +} + +// WrapDBConn @TODO +func WrapDBConn(dbConn *bun.DB, log *logrus.Logger) *DBConn { +	var errProc func(error) db.Error +	switch dbConn.Dialect().Name() { +	case dialect.PG: +		errProc = processPostgresError +	case dialect.SQLite: +		errProc = processSQLiteError +	default: +		panic("unknown dialect name: " + dbConn.Dialect().Name().String()) +	} +	return &DBConn{ +		errProc: errProc, +		log:     log, +		DB:      dbConn, +	} +} + +// ProcessError processes an error to replace any known values with our own db.Error types, +// making it easier to catch specific situations (e.g. no rows, already exists, etc) +func (conn *DBConn) ProcessError(err error) db.Error { +	switch { +	case err == nil: +		return nil +	case err == sql.ErrNoRows: +		return db.ErrNoEntries +	default: +		return conn.errProc(err) +	} +} + +// Exists checks the results of a SelectQuery for the existence of the data in question, masking ErrNoEntries errors +func (conn *DBConn) Exists(ctx context.Context, query *bun.SelectQuery) (bool, db.Error) { +	// Get the select query result +	count, err := query.Count(ctx) + +	// Process error as our own and check if it exists +	switch err := conn.ProcessError(err); err { +	case nil: +		return (count != 0), nil +	case db.ErrNoEntries: +		return false, nil +	default: +		return false, err +	} +} + +// NotExists is the functional opposite of conn.Exists() +func (conn *DBConn) NotExists(ctx context.Context, query *bun.SelectQuery) (bool, db.Error) { +	// Simply inverse of conn.exists() +	exists, err := conn.Exists(ctx, query) +	return !exists, err +} | 
