lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gus Heck (JIRA)" <>
Subject [jira] [Created] (SOLR-11511) Use existing private field in DistributedUpdateProcessor
Date Wed, 18 Oct 2017 23:43:00 GMT
Gus Heck created SOLR-11511:

             Summary: Use existing private field in DistributedUpdateProcessor
                 Key: SOLR-11511
             Project: Solr
          Issue Type: Improvement
      Security Level: Public (Default Security Level. Issues are Public)
          Components: SolrCloud
    Affects Versions: master (8.0)
            Reporter: Gus Heck

The DistributedUpdateProcessor has a private instance field called coreDesc. It is used in
a few places, but most code navigates to CoreDescriptor from the request object instead. 

The fundamental question of this ticket, is this: is there any reason to distrust this field
and do the navigation directly (in which case maybe we get rid of the field instead?) or can
we trust it and thus should use it where we can. Since it is a private field only ever updated
in the constructor, it's not likely to be changing out from under us. The request from which
it is derived is also held in a private final field, so it very much looks to me like this
field should have been final and should be used.

This might or might not be a performance gain (depending on whether or not the compiler can
optimize away something like this already), but it will be a readability and consistency gain
for sure.

Attaching patch to tidy this up shortly...


This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message