Skip to content

SOLR-17855: restore dropwizard-metrics, runtimeOnly#4468

Merged
dsmiley merged 3 commits into
apache:mainfrom
dsmiley:SOLR-17855-fixDropwizardMetricsDep
May 27, 2026
Merged

SOLR-17855: restore dropwizard-metrics, runtimeOnly#4468
dsmiley merged 3 commits into
apache:mainfrom
dsmiley:SOLR-17855-fixDropwizardMetricsDep

Conversation

@dsmiley
Copy link
Copy Markdown
Contributor

@dsmiley dsmiley commented May 25, 2026

https://issues.apache.org/jira/browse/SOLR-17855

Needed at runtime when we embed a ZK server.

Comment thread gradle/libs.versions.toml Outdated
decompose-decompose = { module = "com.arkivanov.decompose:decompose", version.ref = "decompose" }
decompose-extensions-compose = { module = "com.arkivanov.decompose:extensions-compose", version.ref = "decompose" }
dropwizard-metrics-core = { module = "io.dropwizard.metrics:metrics-core", version.ref = "dropwizard-metrics" }
dropwizard-metrics-jetty12-ee10 = { module = "io.dropwizard.metrics:metrics-jetty12-ee10", version.ref = "dropwizard-metrics" }
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

woops... a mistake; I will remove this line

@dsmiley
Copy link
Copy Markdown
Contributor Author

dsmiley commented May 26, 2026

I'm going to merge this tonight if I don't get a review sooner. Our BATS integration tests have been failing and this fixes it.

Copy link
Copy Markdown
Contributor

@janhoy janhoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, perhaps a rephrase of changelog

Comment thread changelog/unreleased/SOLR-17855-dropwizard-removal.yml Outdated
@dsmiley dsmiley merged commit 5b2dcbc into apache:main May 27, 2026
7 checks passed
@dsmiley dsmiley deleted the SOLR-17855-fixDropwizardMetricsDep branch May 27, 2026 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants