[ https://issues.apache.org/jira/browse/SOLR-8499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15087479#comment-15087479 ]
Shawn Heisey commented on SOLR-8499: ------------------------------------ bq. small tweaks to the existing implementation Good idea. If I had known that there was a "numDeletedDocs()" I would have used it. The same deleted calculation is used in LukeRequestHandler, I can update that too. Creating the map with the proper initial size might make it more efficient. I put hasDeletions in because it's there in LukeRequestHandler and seemed like a good idea. I suppose a check for "deletedDocs > 0" is just as effective as a check for hasDeletions. Allocating less heap memory can be a good thing. Now that I'm taking a second look, I am considering adding curly braces and newlines to the "if" statement that is near the end of the additions to lst, purely for readability. The code is perfectly valid as-is, but I find one-line if clauses to be easier to visually parse if they are fully formatted. > Add segmentCount and hasDeletions to SolrIndexSearcher stats > ------------------------------------------------------------ > > Key: SOLR-8499 > URL: https://issues.apache.org/jira/browse/SOLR-8499 > Project: Solr > Issue Type: Improvement > Affects Versions: 5.4 > Reporter: Shawn Heisey > Assignee: Shawn Heisey > Priority: Minor > Attachments: SOLR-8499.patch, SOLR-8499.patch > > > I'd like to add a statistic to the /admin/mbeans (SolrInfoMBeanHandler) > results -- the number of segments in the Lucene index. I think it would be > best to add it to the stats returned by SolrIndexSearcher. Currently that > info is available from /admin/luke (LukeRequestHandler), but I'm finding it > very painful to add another client request to the code where I want to use > this additional info. > As long as I'm fiddling, hasDeletions would be another addition that some > people might find useful. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org