-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Debloat flyway core #3893
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Which version and edition of Flyway are you using?
10.12.0 (OpenSource edition)
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
yes
Enviroment
Flyway-Mysql
Spring Boot 3
Description
While upgrading to 10.12.0 I noticed a lot of unused classes and dependencies are inside flyway
core
.com.google.code.gson:gson
: flyway-core depends on gson: fails if not present #3672 -> required for reporting which is not in usecom.fasterxml.jackson.dataformat:jackson-dataformat-toml
-> Required forTomlUtils
which is never used anywhereorg.flywaydb.core.internal.reports
, parts oforg.flywaydb.core.api.output
(e.g.DashboardRenderer
) and the correspondingassets/report
folderorg.flywaydb.core.internal.database.sqlite
org.flywaydb.core.internal.database.h2
org.flywaydb.core.internal.database.base.TestContainersDatabaseType
org.flywaydb.core.internal.authentication
org.flywaydb.core.internal.license
e.g.EncryptionUtils
org.flywaydb.core.internal.util
e.g.LocalDateTimeSerializer
org.flywaydb.core.internal.license.FlywayTrialExpiredException
I would recommend to remove these classes from the
core
library and move them to the correspoding existing libs or split them into new, dedicated libs.This way the developers can declare/use them if required or at least exclude them in a easier way.
The text was updated successfully, but these errors were encountered: