remove liquibase-log4j2 module

[StevenMassaro]

Remove the liquibase-log4j2 maven module.

The existing module does not work with any recent version of Liquibase, and per previous discussions (see #503) it would be best to remove the code from the log4j2 codebase. Liquibase intends to release and maintain our own implementation of the necessary log4j2 extension.

[ppkarwasz]

@StevenMassaro,

That's great news: log4j-liquibase have been lacking contributors for years. Can you post a message on the [email protected] mailing list, so that we can coordinate the migration of users to the official Liquibase plugin?

[StevenMassaro]

@StevenMassaro,

That's great news: log4j-liquibase have been lacking contributors for years. Can you post a message on the [email protected] mailing list, so that we can coordinate the migration of users to the official Liquibase plugin?

@ppkarwasz , Liquibase has not released their extension for log4j yet. Would that be a prerequisite for removing this code? I was hoping to remove this code since it caused me some confusion on a recent project I was working on, until I realized that it was out of date.

[jvz]

We can do it whenever if it's already outdated. The next release even.

[StevenMassaro]

We can do it whenever if it's already outdated. The next release even.

Excellent! Do you need anything else from me?

[jvz]

Rebase/merge from current please? :)

[StevenMassaro]

Rebase/merge from current please? :)

Done!

[StevenMassaro]

@jvz Is there anything you need from me for this PR before it can be merged?

[jvz]

May be helpful to add a changelog entry for this. We just adopted a simpler system for filing them: https:/apache/logging-log4j2/blob/release-2.x/CHANGELOG.adoc

You can file a GitHub Issue to go with this, too, providing additional details. Otherwise, you can simply link to this PR for the changelog entry.

[StevenMassaro]

@jvz Just added the changelog entry. I saw that the build failed, though to my untrained eye the failures look unrelated to my changes. Let me know if I'm reading that wrong.

[jvz]

Yeah that looks irrelevant, don't worry.

[jvz]

Thanks for the PR!