]> Cypherpunks repositories - gostls13.git/commit
database/sql/driver: document Valuer must not panic
authorDaniel Theophanes <kardianos@gmail.com>
Fri, 5 Apr 2019 17:33:43 +0000 (10:33 -0700)
committerDaniel Theophanes <kardianos@gmail.com>
Thu, 13 Jun 2019 02:49:03 +0000 (02:49 +0000)
commit73abea300a89f0a9b801a4662e3688a5747f1859
tree638ccf09514d3c464ad454243f6968bf618f71fe
parent0b6e3bf4ec1f52a343ab0680ca0347514e33b109
database/sql/driver: document Valuer must not panic

While it is possible for the connection pool to guard against panics
on every valuer read, this seems like a high cost to be added,
in both code complexity and possible runtime performance.

Most uses of the Valuer will be trivial, like returning
a struct field. Optimize for that case. If sometime may panic the
valuer should itself use recover and return an error.

Fixes #26332

Change-Id: Iad18780b8028f669f5a7841b74a5384d62fb6a7e
Reviewed-on: https://go-review.googlesource.com/c/go/+/170700
Reviewed-by: Ian Lance Taylor <iant@golang.org>
src/database/sql/driver/types.go