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

ChangeExecListener-API cannot be used due to HubChangeExecListener #1651

Closed
ckulenkampff opened this issue Jan 26, 2021 · 1 comment · Fixed by #1655
Closed

ChangeExecListener-API cannot be used due to HubChangeExecListener #1651

ckulenkampff opened this issue Jan 26, 2021 · 1 comment · Fixed by #1655

Comments

@ckulenkampff
Copy link

ckulenkampff commented Jan 26, 2021

Environment

Liquibase Version: 4.2.2

Liquibase Integration & Version: Java 1.8

Description

The API for providing a custom ChangeExecListener is broken, because the internal HubChangeExecListener is now obligatory.

Steps To Reproduce

  • Implement a custom ChangeExecListener
  • Provide it to Liquibase through Liquibase.setChangeExecListener(ChangeExecListener)
  • Perform an update like Liquibase.update(String) that directly accesses the database

Actual Behavior

A RuntimeException ChangeExecListener already defined is thrown.

Expected/Desired Behavior

The provided ChangeExecListener is used without an exception. Being able to add multiple ChangeExecListener would be optimal.

Additional Context

Currently I use the ChangeExecListener for adding database specific setup commands when the first changeset is actually executed.

@molivasdat
Copy link
Contributor

Hi @ckulenkampff Thanks for reporting this error. We are looking to have a fix pretty soon on this issue. Stay tuned.

@molivasdat molivasdat linked a pull request Jan 28, 2021 that will close this issue
8 tasks
@sync-by-unito sync-by-unito bot closed this as completed Feb 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants