bricolages / bricolage

AWS-oriented Data Warehouse Framework
114 stars 20 forks source link

Replace DAO connection to one shared connection from three separated connections #121

Closed ragi256 closed 4 years ago

ragi256 commented 4 years ago

DAOのために用意した DAO::Job DAO::JobNet DAO::JobExecution がそれぞれ個別に PostgresConnection を開いていました。

できうる限りbricolage DBとのコネクションを同じものを使い回せるように、同一のdatasourceから張るコネクションを一本に共通化するメソッドを追加し、そちらでコネクションを張るようにしました。 DAOからbricolage DBに接続する際はすべてこの open_shared_connection を利用するようにします。

ragi256 commented 4 years ago

https://github.com/bricolages/bricolage/pull/121/commits/175803149f7d12fdaca536c825ca27349feeb8ee で以下の修正を行いました(commit分割しそびれました)