Skip to content

connect uses of database.DB

Warren Gifford requested to merge cc/remove-database-new into main

Created by: camdencheek

This is another PR that reduces our dependency on database.NewDB() by passing the database.DB type through rather than converting to dbutil.DB and back. The process here is just:

  1. look for a use of database.NewDB()
  2. convert the dbutil.DB it's wrapping into a database.DB
  3. update call sites
  4. repeat

All these uses of database.NewDB() were intended to be temporary until more of the codebase was converted to use database.DB instead of dbutil.DB. It allowed me to make these changes incrementally, but now we're getting to the point where we can connect all the disconnected database.DB instances into one large chain.

The changes here are extremely mechanical, so I apologize for the continued stream of boring PRs 😄

Merge request reports