Clean up instantiation of connection objects

[wasade]

Once #1131 is merged, we should go through and clean up instances in the codebase of:

conn_handler = SQLConnectionHandler() if conn_handler is None else conn_handler

And the variants. Either the conn_handler should be passed in, or a new object created. Since the database connections are not reopened on instantiation (unless necessary), the overhead of just doing SQLConnectionHandler() will be minor

cc @antgonza, who originally recommended opening this

[josenavas]

There might be cases in which that's not possible, but worth doublechecking.

[ElDeveloper]

Is this a bug or an enhancement?

[wasade]

Style/refactor
On Apr 29, 2015 9:05 PM, "Yoshiki Vázquez Baeza" notifications@github.com
wrote:

Is this a bug or an enhancement?

—
Reply to this email directly or view it on GitHub
#1136 (comment).

[ElDeveloper]

Got it, thanks!

On (Apr-29-15|20:10), Daniel McDonald wrote:

Style/refactor
On Apr 29, 2015 9:05 PM, "Yoshiki Vázquez Baeza" notifications@github.com
wrote:

Is this a bug or an enhancement?

—
Reply to this email directly or view it on GitHub
#1136 (comment).

---

Reply to this email directly or view it on GitHub:
#1136 (comment)

[antgonza]

I have already started working on this.

[antgonza]

Closed by #1140