Indeed I was trying to fix any tests using LineFileDocs + MockAnalyzer but clearly I royally screwed up this case because that test case does not use LineFileDocs!!
Sorry :( Thanks for fixing Rob. Mike McCandless http://blog.mikemccandless.com On Thu, Mar 27, 2014 at 5:59 PM, Robert Muir <[email protected]> wrote: > But it makes no sense for this test at all. > > in one field this test indexes 0 terms. in the other field it indexes 1 term. > > and the term is 9 characters long. > > On Thu, Mar 27, 2014 at 2:56 PM, Chris Hostetter > <[email protected]> wrote: >> >> : I don't know why this line was added in r1575538 "don't try to index >> : megaterms". Mike do you know? >> : >> : analyzer.setMaxTokenLength(TestUtil.nextInt(random(), 1, >> : IndexWriter.MAX_TERM_LENGTH)); >> >> I don't know if that specific change in that specific test is a bug or not >> (I haven't looked at MockAnalyzer in years), but i'm pretty sure the >> context of r1575538 as a whole was this progression of failures after >> LUCENE-5472 was committed... >> >> https://mail-archives.apache.org/mod_mbox/lucene-dev/201403.mbox/%3CCAL8Pwkahw=cooln5w7muxq2+8mntzetqzh8dkprk_ibpvd9...@mail.gmail.com%3E >> http://svn.apache.org/r1574917 >> >> https://mail-archives.apache.org/mod_mbox/lucene-dev/201403.mbox/%3CCAL8PwkYAzTsfekHyW4E7Yp6K=sz-f0wdecepu8520pg1o1k...@mail.gmail.com%3E >> http://svn.apache.org/r1575520 >> >> ...i suspect mikemccand was trying to get a jump on other tests that would >> trigger similar problems. >> >> >> -Hoss >> http://www.lucidworks.com/ >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
