[ 
https://issues.apache.org/jira/browse/SOLR-11291?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Christine Poerschke updated SOLR-11291:
---------------------------------------
    Attachment: SOLR-11291.patch

Thanks Omar for updating the patch!

Attached patch is for the {{SolrCore\[Container\]Reporter}} related changes 
only, with small tweaks e.g.
* relocation of the new abstract SolrCore\[Container\]Reporter classes from 
metrics.reporters.solr to metrics package
* addition of class-level javadocs
* removal of unused imports to make 'ant precommit' pass

_(I've removed the {{test/SolrConsoleReporter}} changes from scope since it 
contains many code portions from {{com.codahale.metrics.ConsoleReporter}} and 
both having and maintaining that duplication could be a little tricky. Having 
said that, the {{test/SolrConsoleReporter}} classes remain available here on 
this ticket via the attached patches i.e. developers building their own custom 
metric reporters can still locally apply and use them to explore the workings 
of the Solr metrics reporters.)_

> Adding Solr Core Reporter
> -------------------------
>
>                 Key: SOLR-11291
>                 URL: https://issues.apache.org/jira/browse/SOLR-11291
>             Project: Solr
>          Issue Type: New Feature
>          Components: metrics
>            Reporter: Omar Abdelnabi
>            Assignee: Christine Poerschke
>            Priority: Minor
>         Attachments: SOLR-11291.patch, SOLR-11291.patch, SOLR-11291.patch, 
> SOLR-11291.patch
>
>
> Adds a new reporter, SolrCoreReporter, which allows metrics to be reported on 
> per-core basis.
> Also modifies the SolrMetricManager and SolrCoreMetricManager to take 
> advantage of this new reporter.
> Adds a test/example that uses the  SolrCoreReporter. Also adds randomization 
> to SolrCloudReportersTest.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to