Skip to content
This repository has been archived by the owner on Jun 25, 2021. It is now read-only.

Change Injection of JdbcDatabase #60

Open
noxone opened this issue Jul 16, 2019 · 0 comments
Open

Change Injection of JdbcDatabase #60

noxone opened this issue Jul 16, 2019 · 0 comments

Comments

@noxone
Copy link
Member

noxone commented Jul 16, 2019 •
edited
Loading

The class JdbcDatabase is Closable.
The Guice guidelines suggest to avoid injecting closable resources and that a module should be free of side effects.
Thus Oversigt should change the way the JdbcDatabase instance is to be created.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@noxone