On Thursday, September 27, 2012 3:31:34 AM UTC-7, Christoffer Sawicki wrote:
>
> > This looks good. I would only recommend two changes.
> >
> > 1) Use @db.run(@db.select(1).sql) instead of @db.run("SELECT 1"). Some
> > databases require a dummy FROM clause, and "SELECT 1" is invalid SQL on
> > those databases.
> > 2) Log to all of Sequel's loggers, as opposed to just the first logger.
>
> Fixed!
>
> https://gist.github.com/3793354
>
That looks good. Using the middleware does reduce the concurrency a bit,
requiring a database connection per concurrent web connection, but that's
probably an acceptable price to pay if you are having disconnection issues
and want to silently fix them.
Jeremy
--
You received this message because you are subscribed to the Google Groups
"sequel-talk" group.
To view this discussion on the web visit
https://groups.google.com/d/msg/sequel-talk/-/xwNSpGz5Tu4J.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sequel-talk?hl=en.
