You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "Andrzej Bialecki (Jira)" <ji...@apache.org> on 2019/12/11 12:33:00 UTC

[jira] [Comment Edited] (SOLR-14026) Upgrade Jetty to 9.4.24.v20191120 and dropwizard to 4.1.2

    [ https://issues.apache.org/jira/browse/SOLR-14026?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16993482#comment-16993482 ] 

Andrzej Bialecki edited comment on SOLR-14026 at 12/11/19 12:32 PM:
--------------------------------------------------------------------

{quote}According to the DropWizard docs only one MetricsReporter should be necessary per application
{quote}
Eh, not in our case, since we support different reporting mechanisms and different contexts (jvm, node, core, etc). We also never reset the stats on reading so it's totally safe to have more than 1 reporter accessing the same registry.

Here we should simply pass {{metricManager.registry(registryName)}} to the super constructor. This registry won't be used by this class anyway, because we override the only method that uses it and instead we call the delegate.


was (Author: ab):
We should simply pass {{metricManager.registry(registryName)}} to the super constructor. This registry won't be used by this class anyway, because we override the only method that uses it and instead we call the delegate.

> Upgrade Jetty to 9.4.24.v20191120 and dropwizard to 4.1.2
> ---------------------------------------------------------
>
>                 Key: SOLR-14026
>                 URL: https://issues.apache.org/jira/browse/SOLR-14026
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Erick Erickson
>            Assignee: Erick Erickson
>            Priority: Major
>         Attachments: SOLR-14026.patch
>
>
> Prompted by the linked JIRA.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org