uschindler commented on PR #14275:
URL: https://github.com/apache/lucene/pull/14275#issuecomment-2694748030

   But basically the whole idea here is to allow to replace codecs, which is in 
reality not wanted at all. If you want a different codec, name it differently.
   
   So you am not fully happy with this. The SPI mechanism was never thought to 
replace existing impls at the time or by ordering of classpath. I know we had 
this at beginning for testing purposes on the transition to codecs in Lucene 4, 
but it's an anti-pattern.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to