On 2/11/2013 7:47 PM, Mark Miller wrote:
Doesn't sound right to me. I'd guess you heard wrong.

I did a search for "id" with the quotes throughout the branch_4x source code. After excluding test code, test files, and other things that looked like they have good reason to be hardcoded, I was left with the following:

This class looks like a definite problem.
org.apache.solr.common.cloud.HashBasedRouter
    Object  idObj = sdoc.getFieldValue("id");  // blech

This class uses "id" in a way that looks bad to me, but it's just for logging:
org.apache.solr.client.solrj.impl.ConcurrentUpdateSolrServer

I can't tell if use of "id" in these classes is problematic.
org.apache.solr.handler.admin.LukeRequestHandler
org.apache.solr.handler.admin.QueryElevationComponent
org.apache.solr.handler.component.RealTimeGetComponent
org.apache.solr.handler.loader.JsonLoader
org.apache.solr.handler.loader.XMLLoader
org.apache.solr.spelling.SpellCheckCollator

These classes use "id" in a way that does not look problematic to me, but should be reviewed.
org.apache.solr.cloud.ElectionContext
org.apache.solr.cloud.Overseer
org.apache.solr.cloud.OverseerCollectionProcessor
org.apache.solr.core.JmxMonitoredMap
org.apache.solr.handler.admin.ThreadDumpHandler

Thanks,
Shawn

Reply via email to