[GitHub] [lucene] uschindler commented on pull request #12436: Move max vector dims limit to Codec

["uschindler (via GitHub)" <gi...@apache.org>]

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-unsubscribe@lucene.apache.org

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


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