[
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
