One minor suggestion I have is to rename this attribute to database_engine, just to avoid some confusion (like: if database == "database").