alessandrobenedetti commented on PR #12306: URL: https://github.com/apache/lucene/pull/12306#issuecomment-1552901044
As expressed in the poll, I like this idea, thanks @dsmiley for opening a draft pull request, while we collect more opinions in the poll. My main question for everyone is: how do we decide on a meaningful limit? Why is '1024' reasonable? The reason I am supporting the reconfigurability option is that I would have no idea of a reasonable limit to be enforced on our users. I also doubt we should enforce a limit whatsoever right now, with no technical grounding i.e. there is no data structure or typing choice optimized around the '1024' limit I'll think further about this later on today! -- 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
