Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Application Fails to Start without a Database #11

Open
matthayter opened this issue May 12, 2016 · 3 comments
Open

Application Fails to Start without a Database #11

matthayter opened this issue May 12, 2016 · 3 comments

Comments

@matthayter
Copy link
Contributor

Specifically, if the block in the configuration is missing:

"metrics_clusteragg": {
      "jdbcUrl": "jdbc:h2:/opt/cluster-aggregator/data/metrics:clusteragg;AUTO_SERVER=TRUE;AUTO_SERVER_PORT=7067;MODE=PostgreSQL;INIT=create schema if not exists clusteragg;DB_CLOSE_DELAY=-1",
      "driverName": "org.h2.Driver",
      "username": "sa",
      "password": "secret",
      "maximumPoolSize": 2,
      "minimumIdle": 2,
      "idleTimeout": 0,
      "modelPackages": [ "com.arpnetworking.clusteraggregator.models.ebean" ]
    },

... then the application will not start, throwing a Guice unable-to-initialize type error. This DB (AFAIK) is only used for Circonus Sinks.

Ideally, the application should be able to run without this configuration if no Circonus sinks are in use.

@vjkoskela vjkoskela changed the title Application won't start without configuration for a Circonus-specific database Application Fails to Start without a Database May 12, 2016
@vjkoskela
Copy link
Member

Edited issue title; the database is not Circonus specific, although that was one of the few active use cases for it at Groupon. The database should be lazy initialized and therefore the configuration block only needed if it is used. However, I suspect that you @matthayter will actually want to use this once you have the reporting subsystem completed in order to better monitor your clients. This will require a database. Were there any reasons related to this project that prompted its removal?

@vjkoskela
Copy link
Member

@BrandonArp I believe you made the database optional; can this be closed?

@vjkoskela
Copy link
Member

@BrandonArp have you confirmed this was fixed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants