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

Robert Muir commented on LUCENE-4440:
-------------------------------------

I would prefer something like this patch over the abstract method we currently 
have, especially if we have some way to throw an exception in the
SPI loader if you do a chicken-egg.

Can we make a separate patch to throw an exception if you do this? I definitely 
would support this null check for 4.0, because currently
you will get a confusing exception.
                
> FilterCodec should take a delegate Codec in its ctor
> ----------------------------------------------------
>
>                 Key: LUCENE-4440
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4440
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: core/codecs
>            Reporter: Shai Erera
>            Assignee: Shai Erera
>            Priority: Minor
>         Attachments: LUCENE-4440.patch
>
>
> FilterCodec has a delegate() method through which an extension can return its 
> delegate Codec. This method is called on every Codec method. Adrien, on 
> LUCENE-4391, failed to pass a Codec in the ctor, since he couldn't called 
> Codec.forName().
> Instead, we should just pass e.g. new Lucene40Codec(). I'll post a patch 
> shortly.

--
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