Tighten up logging and error reporting

src/github.com/otoolep/rqlite/command/write_command.go

@@ -25,6 +25,5 @@ func (c *WriteCommand) CommandName() string {
 // Executes an sqlite statement.
 func (c *WriteCommand) Apply(server raft.Server) (interface{}, error) {
 	db := server.Context().(*db.DB)
-	db.Exec(c.Stmt)
+	return nil, db.Execute(c.Stmt)
-	return nil, nil
 }

src/github.com/otoolep/rqlite/db/db.go

@@ -2,6 +2,7 @@ package db
 
 import (
 	"database/sql"
+	"errors"
 	"fmt"
 	"log"
 	"os"
@@ -14,15 +15,20 @@ const (
 	dbName = "db.sqlite"
 )
 
+// Errors
+var RowScanError = errors.New("Row scan failure")
+var QueryExecuteError = errors.New("Query execute error")
+
 // The SQL database.
 type DB struct {
 	dbConn *sql.DB
 }
 
+// Query result types
 type RowResult map[string]string
 type RowResults []map[string]string
 
-// Creates a new database.
+// New creates a new database.
 func New(dir string) *DB {
 	path := path.Join(dir, dbName)
 	os.Remove(path)
@@ -37,11 +43,13 @@ func New(dir string) *DB {
 	}
 }
 
-// Executes the query.
+// Query runs the supplied query against the sqlite database. It returns a slice of
-func (db *DB) Query(query string) RowResults {
+// RowResults.
+func (db *DB) Query(query string) (RowResults, error) {
 	rows, err := db.dbConn.Query(query)
 	if err != nil {
-		fmt.Println(err.Error())
+		log.Fatal("failed to execute query", err.Error())
+		return nil, QueryExecuteError
 	}
 	defer rows.Close()
 
@@ -49,15 +57,16 @@ func (db *DB) Query(query string) RowResults {
 
 	columns, _ := rows.Columns()
 	rawResult := make([][]byte, len(columns))
-	dest := make([]interface{}, len(columns)) // A temporary interface{} slice
+	dest := make([]interface{}, len(columns))
 	for i, _ := range rawResult {
-		dest[i] = &rawResult[i] // Put pointers to each string in the interface slice
+		dest[i] = &rawResult[i] // Pointers to each string in the interface slice
 	}
 
 	for rows.Next() {
 		err = rows.Scan(dest...)
 		if err != nil {
-			log.Fatal("Failed to scan row", err)
+			log.Fatal("failed to scan row", err)
+			return nil, RowScanError
 		}
 
 		r := make(RowResult)
@@ -70,11 +79,11 @@ func (db *DB) Query(query string) RowResults {
 		}
 		results = append(results, r)
 	}
-	return results
+	return results, nil
 }
 
-// Sets the value for a given key.
+// Execute executes the given sqlite statement, of a type that doesn't return rows.
-func (db *DB) Exec(stmt string) {
+func (db *DB) Execute(stmt string) error {
-	_, _ = db.dbConn.Exec(stmt)
+	_, err := db.dbConn.Exec(stmt)
-	return
+	return err
 }

src/github.com/otoolep/rqlite/server/server.go

@@ -164,7 +164,12 @@ func (s *Server) readHandler(w http.ResponseWriter, req *http.Request) {
 		w.WriteHeader(http.StatusBadRequest)
 		return
 	}
-	b, err = json.Marshal(s.db.Query(string(b)))
+
+	r, err := s.db.Query(string(b))
+	if err != nil {
+		http.Error(w, err.Error(), http.StatusInternalServerError)
+	}
+	b, err = json.Marshal(r)
 	if err != nil {
 		http.Error(w, err.Error(), http.StatusInternalServerError)
 		return