Skip to content

Close connection after use in LiquibaseEndpoint #6118

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

Closed

Conversation

joshiste
Copy link
Contributor

@joshiste joshiste commented Jun 5, 2016

Currently (1.3.x) the LiquibaseEndpoint doesn't close the database connection after using it. This PR fixes this.

  • I have signed the CLA

We need to close the JDBC connection after using it, otherwise everytime the
LiquibaseEndpoint is invoked it fetches a connection from the pool without
returning it.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 5, 2016
@philwebb philwebb added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Jun 6, 2016
@philwebb philwebb added this to the 1.3.6 milestone Jun 6, 2016
philwebb added a commit that referenced this pull request Jun 11, 2016
* 1.3.x-liquibase-endpoint:
  Close connection after use in LiquibaseEndpoint
@philwebb philwebb closed this in e89063c Jun 11, 2016
@philwebb
Copy link
Member

Thanks!

@joshiste joshiste deleted the 1.3.x-liquibase-endpoint branch June 11, 2016 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants