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