Odg: Odg: Odg: Odg: Odg: Lucene upgrade

2020-01-07 Thread Mario Kevo
Hi all, Please could someone review #4395<https://github.com/apache/geode/pull/4395>. BR, Mario Šalje: Mario Kevo Poslano: 17. prosinca 2019. 14:30 Prima: Jason Huynh Kopija: geode Predmet: Odg: Odg: Odg: Odg: Odg: Lucene upgrade Hi Jason, Nice ca

Odg: Odg: Odg: Odg: Odg: Lucene upgrade

2019-12-17 Thread Mario Kevo
: Odg: Odg: Odg: Lucene upgrade Hi Mario, I think I see what is going on here. The logic for "reindex" code was a bit off ( it expected reindex features to be complete by a certain release). I have a PR on develop to adjust that calculation (https://github.com/apache/geode/pull/

Re: Odg: Odg: Odg: Odg: Lucene upgrade

2019-12-13 Thread Jason Huynh
In tests it tried to execute a query after all members are rolled. > > BR, > Mario > > -- > *Šalje:* Jason Huynh > *Poslano:* 11. prosinca 2019. 23:08 > *Prima:* Mario Kevo > *Kopija:* geode > *Predmet:* Re: Odg: Odg: Odg: Lucene upgrade

Odg: Odg: Odg: Odg: Lucene upgrade

2019-12-12 Thread Mario Kevo
query after all members are rolled. BR, Mario Šalje: Jason Huynh Poslano: 11. prosinca 2019. 23:08 Prima: Mario Kevo Kopija: geode Predmet: Re: Odg: Odg: Odg: Lucene upgrade Hi Mario, Is the same test failing? If it's a different test, could you tell us

Re: Odg: Odg: Odg: Lucene upgrade

2019-12-11 Thread Jason Huynh
Huynh > *Poslano:* 6. prosinca 2019. 20:53 > *Prima:* Mario Kevo > *Kopija:* geode > *Predmet:* Re: Odg: Odg: Lucene upgrade > > Hi Mario, > > I made a PR against your branch for some of the changes I had to do to get > past the Index too new exception. Summary - r

Odg: Odg: Odg: Lucene upgrade

2019-12-11 Thread Mario Kevo
lower number of entries in tests? BR, Mario Šalje: Jason Huynh Poslano: 6. prosinca 2019. 20:53 Prima: Mario Kevo Kopija: geode Predmet: Re: Odg: Odg: Lucene upgrade Hi Mario, I made a PR against your branch for some of the changes I had to do to get past

Re: Odg: Odg: Lucene upgrade

2019-12-06 Thread Jason Huynh
Hi Mario, I made a PR against your branch for some of the changes I had to do to get past the Index too new exception. Summary - repo creation, even if no writes occur, appear to create some meta data that the old node attempts to read and blow up on. The pr against your branch just prevents the

Odg: Odg: Lucene upgrade

2019-12-06 Thread Mario Kevo
Hi Jason, I tried to upgrade from 6.6.2 to 7.1.0 and got the following exception: org.apache.lucene.index.IndexFormatTooNewException: Format version is not supported (resource BufferedChecksumIndexInput(segments_2)): 7 (needs to be between 4 and 6) It looks like the fix is not good. What I se