[ 
https://issues.apache.org/jira/browse/LUCENENET-90?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12522353
 ] 

George Aroush commented on LUCENENET-90:
----------------------------------------

Hi Digy,

I don't want to take this patch as is.  It is changing the logic of DocCount() 
such that it will now diverge from the Java version and by doing so we may be 
hiding the real issue.  I believe your patch is workaround rather then fixing 
the real issue.  Let us debug this further.

Regards,

-- George

> Nunit test for TestIndexModifier.TestIndex
> ------------------------------------------
>
>                 Key: LUCENENET-90
>                 URL: https://issues.apache.org/jira/browse/LUCENENET-90
>             Project: Lucene.Net
>          Issue Type: Bug
>            Reporter: Digy
>            Priority: Minor
>         Attachments: IndexModifier.patch, IndexWriter.patch, 
> TestIndexModifier.patch
>
>
> IndexModifier uses DocCount of IndexWriter which returns the count of all 
> documents(including deleted ones) unless segments are merged(ex. when 
> explicitely  calling Optimize). Changing the DocCount of IndexWriter effects 
> other parts of the project. So I added a new method 
> "DocCountExcludingDeletedDocs" to IndexWriter and used it in IndexModifier. 
> With this patch Nunit test succeeds.
> I will also add a patch for a new test method "TestIndexModifierDocCount" 
> into the TestIndexModifier.cs

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to