[SOLR-2094] When using a XPathEntityProcessor nested within another entity, the xpathReader isn't reinitilized for each new document

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.4.1
• Fix Version/s: 6.3
• Component/s: contrib - DataImportHandler

Environment

Solr 1.4

• Assignee: Noble Paul
• Reporter: Niall O'Connor
• Created: 27/Aug/10 20:36
• Updated: 09/Nov/16 08:39
• Resolved: 27/Oct/16 09:24

Description

I have a dih config with a SqlEntityProcessor that retrives a table. I then have a sub-entity with the XPathEntityProcessor type, this takes a value from the table as input to parse through an xml doc.
I find that the first document is created correctly, but then the xpathReader of the XPathEntityProcessor does not reinitialize for the following documents so the initial documents input is used.

<dataSource name="hivseqdb" driver="com.mysql.jdbc.Driver"
	   url="l"
           user="hivseqdb" password="hivseqdb" batchSize="1"/>
           
    <dataSource name="xmlFile" type="FileDataSource" />
    
	<document><entity name="Sequence" dataSource="hivseqdb" pk="se_id" query="SELECT * FROM hivseqdb.sequenceentry where se_id != '1'">
			
            <entity name="FMA_Tissue_Hierarchy" 
            		dataSource="xmlFile"
            		pk="fma-id"
            		forEach="/tissue-samples" 
            		processor="XPathEntityProcessor" 
            		url="/opt/hivseqdb/solr/conf/sub_ontology_translated.xml" 
            		stream="true">
                <field column="tissue-antology-parent-path" xpath="/tissue-samples/tissue[@fma-id='${Sequence.sampleTissueCode}']/parent-path"/>
            </entity>

DocBuilder dose call init on the XPathEntityProcessor but there is a conditional in the init method to check if the xpathReader is null:

  public void init(Context context) {
    super.init(context);
    if (xpathReader == null)
      initXpathReader();
    pk = context.getEntityAttribute("pk");
    dataSource = context.getDataSource();
    rowIterator = null;

  }

So the xPathReader is used again and again. Is there away to reinitialize the xPathReader for every document? Or what is the specific design reason for preserving it?