[Lucene.Net] [jira] [Commented] (LUCENENET-426) Mark BaseFragmentsBuilder methods as virtual

["Itamar Syn-Hershko (JIRA)" <ji...@apache.org>]

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

Itamar Syn-Hershko commented on LUCENENET-426:
----------------------------------------------

Apparently that was not enough. I hit a need to override this one too:

protected Field[] GetFields(IndexReader reader, int docId, String fieldName)

Perhaps it'd make sense to make all protected virtual? In Java you can override anything that is not final, so that would be compatible with the original version.

> Mark BaseFragmentsBuilder methods as virtual
> --------------------------------------------
>
>                 Key: LUCENENET-426
>                 URL: https://issues.apache.org/jira/browse/LUCENENET-426
>             Project: Lucene.Net
>          Issue Type: Improvement
>          Components: Lucene.Net Contrib
>    Affects Versions: Lucene.Net 2.9.2, Lucene.Net 2.9.4, Lucene.Net 3.x, Lucene.Net 2.9.4g
>            Reporter: Itamar Syn-Hershko
>            Priority: Minor
>             Fix For: Lucene.Net 2.9.4, Lucene.Net 2.9.4g
>
>         Attachments: fvh.patch
>
>
> Without marking methods in BaseFragmentsBuilder as virtual, it is meaningless to have FragmentsBuilder deriving from a class named "Base", since most of its functionality cannot be overridden. Attached is a patch for marking the important methods virtual.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira