Don't use a servername as static text in your methods.

I see always a lot of things like this:

var dataset = databaseManager.getDataSetByQuery(‘mydatabaseserver’,query, args, maxReturnedRows);

this completely breaks portability because you suddenly really need to have the same servername everywhere… Always use controller.getServerName() if possible

var dataset = databaseManager.getDataSetByQuery(controller.getServerName(),query, args, maxReturnedRows);

if this is somehow not possible for what every reason then still do something like this:

var dataset = databaseManager.getDataSetByQuery(globals.myservername,query, args, maxReturnedRows);

and configure youre global in a startup script (so it is only done in one place)