[GitHub] [lucene] rmuir commented on pull request #211: LUCENE-5595: re-enable TestICUNormalizer2CharFilter random test, splitting by mode

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

rmuir commented on pull request #211:
URL: https://github.com/apache/lucene/pull/211#issuecomment-879535535


   cc: @magibney 
   
   This is the basic random test that we've had disabled for years. Honestly original bugs could have been in ICU itself, not sure. Maybe the new tests will fail! But I think it is much better for us to enable it in `main` branch with the new gradle build, with tests corresponding to different normalization modes. Maybe we stand a better chance to fix any failures this way.


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