> On May 18, 2016, 1 a.m., Dan Smith wrote: > > geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/LuceneServiceImplIntegrationTest.java, > > line 352 > > <https://reviews.apache.org/r/47496/diff/1/?file=1386143#file1386143line352> > > > > I'm actually surprised this works, since the field names are "title" > > and "description", not "text".
Looked into this a bit, apparently I need to use description:"\hello world\" as the string, otherwise lucene will look for the word "world" in the default field which was the reason why it was working the way it was. In our case the default I think are all fields or whatever the QueryParser is passed - Jason ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47496/#review133671 ----------------------------------------------------------- On May 17, 2016, 10:54 p.m., Jason Huynh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47496/ > ----------------------------------------------------------- > > (Updated May 17, 2016, 10:54 p.m.) > > > Review request for geode, anilkumar gingade, Barry Oglesby, nabarun nag, Dan > Smith, and xiaojian zhou. > > > Repository: geode > > > Description > ------- > > Added integration tests to test three scenarios: > 1.) the lucene indexes are not updated during an in flight transaction (begin > but before rollback or commit) > 2.) the lucene indexes are updated after a commit > 3.) the lucene indexes are not changed after a rollback > > todo: > Do we want to add a dunit test for multiple vms and threads doing > transactions? I avoided adding this test for now as I thought that was more > of a "testing the geode transaction feature" rather than testing how the > feature affects lucene. > > > Diffs > ----- > > > geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/internal/LuceneServiceImplIntegrationTest.java > fa3392c > > Diff: https://reviews.apache.org/r/47496/diff/ > > > Testing > ------- > > > Thanks, > > Jason Huynh > >
