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

Initialize the repository on servlet load, or clustering may get unhappy... #13

Merged
merged 2 commits into from Jan 20, 2013

Conversation

cbeer
Copy link
Contributor

@cbeer cbeer commented Jan 19, 2013

..... i think

Am I off-base here? Is there a better way to get the repository initialized on the servlet startup (Spring?)?

@ajs6f
Copy link
Contributor

ajs6f commented Jan 19, 2013

I'm definitely cool with eager initialization... if we expect to move to a DI framework (Spring, Blueprint, whatever) I'm inclined to say we do so earlier rather than writing a lot of code we won't keep. See here:

https://community.jboss.org/message/792765

for a brief discussion of Spring.

Alternatively, we could go straight for the high ground and use the OSGi Service Framework... I'd happy to cook that up, as soon as I get this stupid FOXML sequencer working.

ajs6f added a commit that referenced this pull request Jan 20, 2013
Initialize the repository on servlet load, or clustering may get unhappy...

We will likely go on to DI to deal with this kind of problem, but let's start with what we've got!
@ajs6f ajs6f merged commit 7fd73db into master Jan 20, 2013
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

Successfully merging this pull request may close these issues.

None yet

2 participants