[GitHub] [lucene-solr] mikemccand commented on a change in pull request #2080: LUCENE-8947: Skip field length accumulation when norms are disabled

[GitBox <gi...@apache.org>]

mikemccand commented on a change in pull request #2080:
URL: https://github.com/apache/lucene-solr/pull/2080#discussion_r523259786



##########
File path: lucene/core/src/test/org/apache/lucene/index/TestCustomTermFreq.java
##########
@@ -458,4 +458,50 @@ public void testFieldInvertState() throws Exception {
 
     IOUtils.close(w, dir);
   }
+
+  // LUCENE-8947: Indexing fails with "too many tokens for field" when using custom term frequencies

Review comment:
       Maybe add `when using large enough custom term frequencies to overflow int on accumulation`?




----------------------------------------------------------------
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.

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