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

Async liquibase #526

Closed
wants to merge 1 commit into from
Closed

Async liquibase #526

wants to merge 1 commit into from

Conversation

levice14
Copy link
Collaborator

fixes #294 (?)

Signed-off-by: Bonczidai Levente [email protected]

Signed-off-by: Bonczidai Levente <[email protected]>
@levice14
Copy link
Collaborator Author

committed a poc - if you activate the mechanism (uncomment the dataSource and liquibase beans in config ) it gives exception at startup

hibernate gives the exception, it tries to access the DB but it is not ready since liquibase is doing it on another thread - seems to me that everyhing is dependent on everything here (can we do it parallel?) - anyway the time used by liquibase is around 1,5 sec (not 7-8 like in the example)

I am not sure there is an easy way to achieve async loading - do you think otherwise?

@oritstone @meshipeer @orius123 @meirwah

@levice14 levice14 added this to the 1.0 - sprint 1 milestone Dec 17, 2015
@levice14 levice14 self-assigned this Dec 17, 2015
@orius123
Copy link
Contributor

It wasn't supposed to solve all of the 8 sec, just the liquibase part, anyways I think it's a bad approach to start rolling threads for loading beans.
the right solution is to get rid of the DB in score, or to start it as a daemon.

@levice14 levice14 closed this Jan 5, 2016
@levice14 levice14 deleted the async-liquibase branch January 5, 2016 10:47
@levice14 levice14 removed this from the 1.0 - sprint 1 milestone Jan 5, 2016
@levice14 levice14 removed their assignment Jan 5, 2016
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.

More then 15 sec for CLI to start on Ubuntu - version 0.7.29 [3]
3 participants