uschindler commented on PR #12436: URL: https://github.com/apache/lucene/pull/12436#issuecomment-1653475761
This looks fine to me. I am happy that we do not have stupid system properties. Somebody who wants to raise the limit (or lower it to 32 like in the test) can simply implement an own codec. One question I have: What happens if you open an index with a higher limit in field infos and you use default codec? I think this is unsupported, but in that case the implementor of the codec should possibly use own codec name. -- 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
