Merge pull request #675 from pachyderm/derekchiang-patch-1

Fix a bug where error is not returned when primary key is not found
pachyderm · Jul 22, 2016 · 150e0a7 · 150e0a7
2 parents 6efecc8 + b5e7def
commit 150e0a7
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/server/pps/persist/server/rethink_api_server.go b/src/server/pps/persist/server/rethink_api_server.go
@@ -502,7 +502,7 @@ func (a *rethinkAPIServer) getMessageByPrimaryKey(table Table, key interface{},
 	if cursor.IsNil() {
 		return fmt.Errorf("%v %v not found", table, key)
 	}
-	if cursor.Next(message) {
+	if !cursor.Next(message) {
 		return cursor.Err()
 	}
 	return nil