[ 
https://issues.apache.org/jira/browse/SOLR-4508?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Vadim Kirilchuk updated SOLR-4508:
----------------------------------

    Description: 
Hi, 

trunk/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/config/DIHConfiguration.java
trunk/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/config/Document.java

I see that there are two classes in dih codebase which are quite similar, and 
looks like the latter (Document.java) has no usages.
Is it dead code? Should Document.java be removed?

Thanks

  was:
Hi, 

I see that there are two classes in dih codebase which are quite similar, and 
looks like the latter (Document.java) has no usages.

trunk/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/config/Document.java

Is it dead code? Should it be removed?

Thanks

    
> DIHConfiguration and Document classes are suspiciously similar
> --------------------------------------------------------------
>
>                 Key: SOLR-4508
>                 URL: https://issues.apache.org/jira/browse/SOLR-4508
>             Project: Solr
>          Issue Type: Improvement
>          Components: contrib - DataImportHandler
>            Reporter: Vadim Kirilchuk
>
> Hi, 
> trunk/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/config/DIHConfiguration.java
> trunk/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/config/Document.java
> I see that there are two classes in dih codebase which are quite similar, and 
> looks like the latter (Document.java) has no usages.
> Is it dead code? Should Document.java be removed?
> Thanks

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to