Hmm that doesn't fix it. The magical seed is: 50686536365145364L (for TRR2).
I still see the IAE even if I remove that RegExp.NONE. Mike On Mon, Jul 26, 2010 at 1:28 PM, Robert Muir <[email protected]> wrote: > I don't think we should do that. I think i found the problem: > AutomatonTestUtil's randomRegexp does this: > try { > return new RegExp(regexp, RegExp.NONE); > } catch (Exception e) {} > I think the RegExp.NONE is the problem, and we should remove it. > because the test then toString's it and compiles it again, but without this > option. > On Mon, Jul 26, 2010 at 1:16 PM, Michael McCandless > <[email protected]> wrote: >> >> My random stress testing hit an IllegalArgExc because the random >> regexp was malformed. >> >> Does this patch look OK to fix? >> >> Index: src/test/org/apache/lucene/search/TestRegexpRandom2.java >> =================================================================== >> --- src/test/org/apache/lucene/search/TestRegexpRandom2.java (revision >> 979227) >> +++ src/test/org/apache/lucene/search/TestRegexpRandom2.java (working >> copy) >> @@ -130,6 +130,14 @@ >> * simple regexpquery implementation. >> */ >> private void assertSame(String regexp) throws IOException { >> + try { >> + new RegExp(regexp); >> + } catch (IllegalArgumentException iae) { >> + // the random regexp could be malformed, eg "foo"bar", >> + // so we ignore this >> + return; >> + } >> + >> RegexpQuery smart = new RegexpQuery(new Term("field", regexp)); >> DumbRegexpQuery dumb = new DumbRegexpQuery(new Term("field", regexp)); >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> > > > > -- > Robert Muir > [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
