dweiss commented on code in PR #12464:
URL: https://github.com/apache/lucene/pull/12464#discussion_r1276354832
##########
lucene/analysis/common/src/java/org/apache/lucene/analysis/hunspell/WordStorage.java:
##########
@@ -390,7 +391,7 @@ private int flushGroup() throws IOException {
if (++chainLengths[hash] > 20) {
throw new RuntimeException(
- "Too many collisions, please report this to
d...@lucene.apache.org");
Review Comment:
I wonder whether this message should remain as it was. Most people will use
it out of the box and we can then count on their feedback if something is not
right. Those who do override the default will certainly know where to look (we
can report the hashFactor as part of the exception message, perhaps)?
--
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
