janhoy commented on code in PR #4326:
URL: https://github.com/apache/solr/pull/4326#discussion_r3134023841
##########
solr/modules/langid/src/test/org/apache/solr/update/processor/LangDetectLanguageIdentifierUpdateProcessorFactoryTest.java:
##########
@@ -37,8 +41,22 @@ protected SolrInputDocument tooShortDoc() {
return doc;
}
- /* we don't return 'un' for the super-short one (this detector things
hungarian?).
- * replace this with japanese
+ /**
+ * Override the base class test with LangDetect-specific text samples.
+ *
+ * <p>Two differences from the base class:
+ *
+ * <ul>
+ * <li>The "too short" test case is replaced with Japanese (this detector
returns Hungarian for
+ * short ambiguous text rather than "un").
+ * <li>The Russian text is replaced with a cleaner Cyrillic-only sample.
The base class uses a
Review Comment:
No we don’t. It belongs more in a PR comment I believe.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
