[
https://issues.apache.org/jira/browse/SOLR-1000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12669662#action_12669662
]
Shalin Shekhar Mangar commented on SOLR-1000:
---------------------------------------------
First the ClassCastException was because AbstractDataImportHandlerTest tries to
read a string from the attributes map. But in this case, testRecursive put in a
boolean true rather than a string to the 'recursive' attribute. That was fixed
by adding string "true" instead of a boolean. I'll fix
AbstractDataImportHandlerTest to read String.valueOf to handle these cases in
the future.
After this fix, the assert at the end of the testRecursive failed. This is
because it expects to find 3 files but "a.xml", "b.xml" and "c.props" are in
the same directory and due to the 'fileName' regex, c.props won't be picked up.
I guess you meant to add c.props to another child directory inside the one you
are creating?
> DIH FileListEntityProcessor fileName filters directory names and stops
> recursion
> ---------------------------------------------------------------------------------
>
> Key: SOLR-1000
> URL: https://issues.apache.org/jira/browse/SOLR-1000
> Project: Solr
> Issue Type: Improvement
> Components: contrib - DataImportHandler
> Affects Versions: 1.3
> Reporter: Fergus McMenemie
> Assignee: Shalin Shekhar Mangar
> Attachments: SOLR-1000.patch
>
> Original Estimate: 24h
> Remaining Estimate: 24h
>
> I have been trying to find out why DIH in FileListEntityProcessor mode did
> not appear to be recursing into subdirectories. Going through
> FileListEntityProcessor.java I eventually tumbled to the fact that my
> filename filter setting from data-config.xml also applied to directory names.
> Now, I feel that the fieldName filter should be applied to files fed into the
> parser, it should not be applied to the directory names we are recursing
> through. I bodged the code to adjust the behavior so that the "FileName" and
> "excludes" attributes of "entity" only apply to filenames and not directory
> names. It now recurses though my directory tree only indexing the appropriate
> files! I think the new behavior is more standard.
> I will submit the a patch once I have constructed one!
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
