I see this PR makes it safer to Ping the db, even when a txn has failed. But I don't fully grok it. Specifically, I'm wondering how a caller can be certain that rows aren't nil? It looks like here that simpleQuery could return nil rows and a nil error. Is my understanding incorrect?

Thanks