Re: Release 6.6

2017-06-07 Thread Ishan Chattopadhyaya
 >>>>>>>>>
>> >>>>>>>>> I wish to backport a fix from SOLR-8440 (last commit) to the
>> >>>>>>>>> release branch. It affects only the feature introduced in
>> SOLR-8440. Please
>> >>>>>>>>> let me know if someone has any objections.
>> >>>>>>>>>
>> >>>>>>>>> Also, I'm planning to build the RC in another 3-4 hours. Please
>> let
>> >>>>>>>>> me know if there's something that someone is working on which
>> needs to get
>> >>>>>>>>> in before that.
>> >>>>>>>>>
>> >>>>>>>>> Thanks and regards,
>> >>>>>>>>> Ishan
>> >>>>>>>>>
>> >>>>>>>>>
>> >>>>>>>>> On Sun, May 14, 2017 at 5:02 PM, Ishan Chattopadhyaya
>> >>>>>>>>> <ichattopadhy...@gmail.com> wrote:
>> >>>>>>>>>>
>> >>>>>>>>>> Sure Steve! Thanks!
>> >>>>>>>>>>
>> >>>>>>>>>> On Sun, May 14, 2017 at 2:34 PM, Steve Rowe <sar...@gmail.com>
>> >>>>>>>>>> wrote:
>> >>>>>>>>>>>
>> >>>>>>>>>>> Ishan,
>> >>>>>>>>>>>
>> >>>>>>>>>>> Okay to include https://issues.apache.org/jira
>> /browse/LUCENE-7821
>> >>>>>>>>>>> for 6.6?
>> >>>>>>>>>>>
>> >>>>>>>>>>> --
>> >>>>>>>>>>> Steve
>> >>>>>>>>>>> www.lucidworks.com
>> >>>>>>>>>>>
>> >>>>>>>>>>> > On May 12, 2017, at 12:51 PM, jim ferenczi
>> >>>>>>>>>>> > <jim.feren...@gmail.com> wrote:
>> >>>>>>>>>>> >
>> >>>>>>>>>>> > Ok thanks Ishan.
>> >>>>>>>>>>> >
>> >>>>>>>>>>> > Le 12 mai 2017 18:44, "Ishan Chattopadhyaya"
>> >>>>>>>>>>> > <ichattopadhy...@gmail.com> a écrit :
>> >>>>>>>>>>> > Sure, Jim. Please go ahead!
>> >>>>>>>>>>> >
>> >>>>>>>>>>> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi
>> >>>>>>>>>>> > <jim.feren...@gmail.com> wrote:
>> >>>>>>>>>>> > Hi,
>> >>>>>>>>>>> > Would be great to have
>> >>>>>>>>>>> > https://issues.apache.org/jira/browse/LUCENE-7824 included
>> for 6.6. Can I
>> >>>>>>>>>>> > merge the fix this week end Ishan ?
>> >>>>>>>>>>> >
>> >>>>>>>>>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya
>> >>>>>>>>>>> > <ichattopadhy...@gmail.com>:
>> >>>>>>>>>>> > Done, Adrien. Thanks!
>> >>>>>>>>>>> >
>> >>>>>>>>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand
>> >>>>>>>>>>> > <jpou...@gmail.com> wrote:
>> >>>>>>>>>>> > Ishan, wdyt about running addVersion on the
>> branch_6x/master as
>> >>>>>>>>>>> > well? I think it would help realize that the 6.6 branch has
>> been cut when
>> >>>>>>>>>>> > looking at the CHANGES.txt file and not forget about
>> backporting?
>> >>>>>>>>>>> >
>> >>>>>>>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya
>> >>>>>>>>>>> > <ichattopadhy...@gmail.com> a écrit :
>> >>>>>>>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free
>> to
>> >>>>>>>>>>&g

Re: Release 6.6

2017-06-07 Thread Mikhail Khludnev
t;>>>>>>> Thanks and regards,
> >>>>>>>>> Ishan
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> On Sun, May 14, 2017 at 5:02 PM, Ishan Chattopadhyaya
> >>>>>>>>> <ichattopadhy...@gmail.com> wrote:
> >>>>>>>>>>
> >>>>>>>>>> Sure Steve! Thanks!
> >>>>>>>>>>
> >>>>>>>>>> On Sun, May 14, 2017 at 2:34 PM, Steve Rowe <sar...@gmail.com>
> >>>>>>>>>> wrote:
> >>>>>>>>>>>
> >>>>>>>>>>> Ishan,
> >>>>>>>>>>>
> >>>>>>>>>>> Okay to include https://issues.apache.org/
> jira/browse/LUCENE-7821
> >>>>>>>>>>> for 6.6?
> >>>>>>>>>>>
> >>>>>>>>>>> --
> >>>>>>>>>>> Steve
> >>>>>>>>>>> www.lucidworks.com
> >>>>>>>>>>>
> >>>>>>>>>>> > On May 12, 2017, at 12:51 PM, jim ferenczi
> >>>>>>>>>>> > <jim.feren...@gmail.com> wrote:
> >>>>>>>>>>> >
> >>>>>>>>>>> > Ok thanks Ishan.
> >>>>>>>>>>> >
> >>>>>>>>>>> > Le 12 mai 2017 18:44, "Ishan Chattopadhyaya"
> >>>>>>>>>>> > <ichattopadhy...@gmail.com> a écrit :
> >>>>>>>>>>> > Sure, Jim. Please go ahead!
> >>>>>>>>>>> >
> >>>>>>>>>>> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi
> >>>>>>>>>>> > <jim.feren...@gmail.com> wrote:
> >>>>>>>>>>> > Hi,
> >>>>>>>>>>> > Would be great to have
> >>>>>>>>>>> > https://issues.apache.org/jira/browse/LUCENE-7824 included
> for 6.6. Can I
> >>>>>>>>>>> > merge the fix this week end Ishan ?
> >>>>>>>>>>> >
> >>>>>>>>>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya
> >>>>>>>>>>> > <ichattopadhy...@gmail.com>:
> >>>>>>>>>>> > Done, Adrien. Thanks!
> >>>>>>>>>>> >
> >>>>>>>>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand
> >>>>>>>>>>> > <jpou...@gmail.com> wrote:
> >>>>>>>>>>> > Ishan, wdyt about running addVersion on the branch_6x/master
> as
> >>>>>>>>>>> > well? I think it would help realize that the 6.6 branch has
> been cut when
> >>>>>>>>>>> > looking at the CHANGES.txt file and not forget about
> backporting?
> >>>>>>>>>>> >
> >>>>>>>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya
> >>>>>>>>>>> > <ichattopadhy...@gmail.com> a écrit :
> >>>>>>>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free
> to
> >>>>>>>>>>> > add bugfixes to the branch, if needed.
> >>>>>>>>>>> > Planning to build the first RC on 15 May. Please let me know
> if
> >>>>>>>>>>> > there are any objections.
> >>>>>>>>>>> >
> >>>>>>>>>>> > Thanks,
> >>>>>>>>>>> > Ishan
> >>>>>>>>>>> >
> >>>>>>>>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya
> >>>>>>>>>>> > <ichattopadhy...@gmail.com> wrote:
> >>>>>>>>>>> > Planning to cut the release branch in another 10-12 hours.
> >>>>>>>>>>> >
> >>>>>>>>>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty
> >>>>>>>>>>> > <mgai...@hotmail.c

Re: Release 6.6

2017-06-07 Thread Erick Erickson
>>>>
>>>>>>>>>>> Ishan,
>>>>>>>>>>>
>>>>>>>>>>> Okay to include https://issues.apache.org/jira/browse/LUCENE-7821
>>>>>>>>>>> for 6.6?
>>>>>>>>>>>
>>>>>>>>>>> --
>>>>>>>>>>> Steve
>>>>>>>>>>> www.lucidworks.com
>>>>>>>>>>>
>>>>>>>>>>> > On May 12, 2017, at 12:51 PM, jim ferenczi
>>>>>>>>>>> > <jim.feren...@gmail.com> wrote:
>>>>>>>>>>> >
>>>>>>>>>>> > Ok thanks Ishan.
>>>>>>>>>>> >
>>>>>>>>>>> > Le 12 mai 2017 18:44, "Ishan Chattopadhyaya"
>>>>>>>>>>> > <ichattopadhy...@gmail.com> a écrit :
>>>>>>>>>>> > Sure, Jim. Please go ahead!
>>>>>>>>>>> >
>>>>>>>>>>> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi
>>>>>>>>>>> > <jim.feren...@gmail.com> wrote:
>>>>>>>>>>> > Hi,
>>>>>>>>>>> > Would be great to have
>>>>>>>>>>> > https://issues.apache.org/jira/browse/LUCENE-7824 included for 
>>>>>>>>>>> > 6.6. Can I
>>>>>>>>>>> > merge the fix this week end Ishan ?
>>>>>>>>>>> >
>>>>>>>>>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya
>>>>>>>>>>> > <ichattopadhy...@gmail.com>:
>>>>>>>>>>> > Done, Adrien. Thanks!
>>>>>>>>>>> >
>>>>>>>>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand
>>>>>>>>>>> > <jpou...@gmail.com> wrote:
>>>>>>>>>>> > Ishan, wdyt about running addVersion on the branch_6x/master as
>>>>>>>>>>> > well? I think it would help realize that the 6.6 branch has been 
>>>>>>>>>>> > cut when
>>>>>>>>>>> > looking at the CHANGES.txt file and not forget about backporting?
>>>>>>>>>>> >
>>>>>>>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya
>>>>>>>>>>> > <ichattopadhy...@gmail.com> a écrit :
>>>>>>>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to
>>>>>>>>>>> > add bugfixes to the branch, if needed.
>>>>>>>>>>> > Planning to build the first RC on 15 May. Please let me know if
>>>>>>>>>>> > there are any objections.
>>>>>>>>>>> >
>>>>>>>>>>> > Thanks,
>>>>>>>>>>> > Ishan
>>>>>>>>>>> >
>>>>>>>>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya
>>>>>>>>>>> > <ichattopadhy...@gmail.com> wrote:
>>>>>>>>>>> > Planning to cut the release branch in another 10-12 hours.
>>>>>>>>>>> >
>>>>>>>>>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty
>>>>>>>>>>> > <mgai...@hotmail.com> wrote:
>>>>>>>>>>> > i was wondering if there was a specific test for
>>>>>>>>>>> > SpanPayloadCheckQuery method
>>>>>>>>>>> >
>>>>>>>>>>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>>>>>>>> >
>>>>>>>>>>> >
>>>>>>>>>>> >
>>>>>>>>>>> > the only implementation i could locate was in collectLeaf of
>>>>>>>>>>> > SpanPayloadCheckQuery
>>>>>>>>>>> >
>>>>>>>>>>> >
>>>>>>>>>>> > I will submit JIRA with Patch
>>>>>>>>>>&g

Re: Release 6.6

2017-06-07 Thread Ishan Chattopadhyaya
M, jim ferenczi <
>>>>>>>>>> jim.feren...@gmail.com> wrote:
>>>>>>>>>> >
>>>>>>>>>> > Ok thanks Ishan.
>>>>>>>>>> >
>>>>>>>>>> > Le 12 mai 2017 18:44, "Ishan Chattopadhyaya" <
>>>>>>>>>> ichattopadhy...@gmail.com> a écrit :
>>>>>>>>>> > Sure, Jim. Please go ahead!
>>>>>>>>>> >
>>>>>>>>>> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <
>>>>>>>>>> jim.feren...@gmail.com> wrote:
>>>>>>>>>> > Hi,
>>>>>>>>>> > Would be great to have https://issues.apache.org/
>>>>>>>>>> jira/browse/LUCENE-7824 included for 6.6. Can I merge the fix
>>>>>>>>>> this week end Ishan ?
>>>>>>>>>> >
>>>>>>>>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <
>>>>>>>>>> ichattopadhy...@gmail.com>:
>>>>>>>>>> > Done, Adrien. Thanks!
>>>>>>>>>> >
>>>>>>>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <
>>>>>>>>>> jpou...@gmail.com> wrote:
>>>>>>>>>> > Ishan, wdyt about running addVersion on the branch_6x/master as
>>>>>>>>>> well? I think it would help realize that the 6.6 branch has been cut 
>>>>>>>>>> when
>>>>>>>>>> looking at the CHANGES.txt file and not forget about backporting?
>>>>>>>>>> >
>>>>>>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>>>>>>>>> ichattopadhy...@gmail.com> a écrit :
>>>>>>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to
>>>>>>>>>> add bugfixes to the branch, if needed.
>>>>>>>>>> > Planning to build the first RC on 15 May. Please let me know if
>>>>>>>>>> there are any objections.
>>>>>>>>>> >
>>>>>>>>>> > Thanks,
>>>>>>>>>> > Ishan
>>>>>>>>>> >
>>>>>>>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>>>>>>>>> ichattopadhy...@gmail.com> wrote:
>>>>>>>>>> > Planning to cut the release branch in another 10-12 hours.
>>>>>>>>>> >
>>>>>>>>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <
>>>>>>>>>> mgai...@hotmail.com> wrote:
>>>>>>>>>> > i was wondering if there was a specific test for
>>>>>>>>>> SpanPayloadCheckQuery method
>>>>>>>>>> >
>>>>>>>>>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>>>>>>> >
>>>>>>>>>> >
>>>>>>>>>> >
>>>>>>>>>> > the only implementation i could locate was in collectLeaf of
>>>>>>>>>> SpanPayloadCheckQuery
>>>>>>>>>> >
>>>>>>>>>> >
>>>>>>>>>> > I will submit JIRA with Patch
>>>>>>>>>> >
>>>>>>>>>> >
>>>>>>>>>> > as a CS *dinosaur* I am more familiar with LISP for dissecting
>>>>>>>>>> sentence fragments (what we called phenomes) than current SEO
>>>>>>>>>> implementations
>>>>>>>>>> >
>>>>>>>>>> >
>>>>>>>>>> > Can you suggest a book to read to better understand Lucenes
>>>>>>>>>> pattern dissection and match algorithms?
>>>>>>>>>> >
>>>>>>>>>> >
>>>>>>>>>> > Many Thanks for helpful guidance
>>>>>>>>>> > Martin
>>>>>>>>>> > __
>>>>>>>>>> 

Re: Release 6.6

2017-06-07 Thread Adrien Grand
>>>>>> >
>>>>>>>>> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <
>>>>>>>>> jim.feren...@gmail.com> wrote:
>>>>>>>>> > Hi,
>>>>>>>>> > Would be great to have
>>>>>>>>> https://issues.apache.org/jira/browse/LUCENE-7824 included for
>>>>>>>>> 6.6. Can I merge the fix this week end Ishan ?
>>>>>>>>> >
>>>>>>>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <
>>>>>>>>> ichattopadhy...@gmail.com>:
>>>>>>>>> > Done, Adrien. Thanks!
>>>>>>>>> >
>>>>>>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com>
>>>>>>>>> wrote:
>>>>>>>>> > Ishan, wdyt about running addVersion on the branch_6x/master as
>>>>>>>>> well? I think it would help realize that the 6.6 branch has been cut 
>>>>>>>>> when
>>>>>>>>> looking at the CHANGES.txt file and not forget about backporting?
>>>>>>>>> >
>>>>>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>>>>>>>> ichattopadhy...@gmail.com> a écrit :
>>>>>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to
>>>>>>>>> add bugfixes to the branch, if needed.
>>>>>>>>> > Planning to build the first RC on 15 May. Please let me know if
>>>>>>>>> there are any objections.
>>>>>>>>> >
>>>>>>>>> > Thanks,
>>>>>>>>> > Ishan
>>>>>>>>> >
>>>>>>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>>>>>>>> ichattopadhy...@gmail.com> wrote:
>>>>>>>>> > Planning to cut the release branch in another 10-12 hours.
>>>>>>>>> >
>>>>>>>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <
>>>>>>>>> mgai...@hotmail.com> wrote:
>>>>>>>>> > i was wondering if there was a specific test for
>>>>>>>>> SpanPayloadCheckQuery method
>>>>>>>>> >
>>>>>>>>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > the only implementation i could locate was in collectLeaf of
>>>>>>>>> SpanPayloadCheckQuery
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > I will submit JIRA with Patch
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > as a CS *dinosaur* I am more familiar with LISP for dissecting
>>>>>>>>> sentence fragments (what we called phenomes) than current SEO
>>>>>>>>> implementations
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > Can you suggest a book to read to better understand Lucenes
>>>>>>>>> pattern dissection and match algorithms?
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > Many Thanks for helpful guidance
>>>>>>>>> > Martin
>>>>>>>>> > __
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > From: Erik Hatcher <erik.hatc...@gmail.com>
>>>>>>>>> > Sent: Sunday, April 30, 2017 2:06 PM
>>>>>>>>> >
>>>>>>>>> > To: dev@lucene.apache.org
>>>>>>>>> > Subject: Re: Release 6.6
>>>>>>>>> >
>>>>>>>>> > Martin -
>>>>>>>>> >
>>>>>>>>> > I have to admit to still being unsure what the gist is here - is
>>>>>>>>> there a bug?   Apologies for not catching what you’re saying/showi

Re: Release 6.6

2017-06-05 Thread Michael McCandless
gt;>>>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <
>>>>>>>> ichattopadhy...@gmail.com>:
>>>>>>>> > Done, Adrien. Thanks!
>>>>>>>> >
>>>>>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com>
>>>>>>>> wrote:
>>>>>>>> > Ishan, wdyt about running addVersion on the branch_6x/master as
>>>>>>>> well? I think it would help realize that the 6.6 branch has been cut 
>>>>>>>> when
>>>>>>>> looking at the CHANGES.txt file and not forget about backporting?
>>>>>>>> >
>>>>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>>>>>>> ichattopadhy...@gmail.com> a écrit :
>>>>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to
>>>>>>>> add bugfixes to the branch, if needed.
>>>>>>>> > Planning to build the first RC on 15 May. Please let me know if
>>>>>>>> there are any objections.
>>>>>>>> >
>>>>>>>> > Thanks,
>>>>>>>> > Ishan
>>>>>>>> >
>>>>>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>>>>>>> ichattopadhy...@gmail.com> wrote:
>>>>>>>> > Planning to cut the release branch in another 10-12 hours.
>>>>>>>> >
>>>>>>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <
>>>>>>>> mgai...@hotmail.com> wrote:
>>>>>>>> > i was wondering if there was a specific test for
>>>>>>>> SpanPayloadCheckQuery method
>>>>>>>> >
>>>>>>>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>>>>> >
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > the only implementation i could locate was in collectLeaf of
>>>>>>>> SpanPayloadCheckQuery
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > I will submit JIRA with Patch
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > as a CS *dinosaur* I am more familiar with LISP for dissecting
>>>>>>>> sentence fragments (what we called phenomes) than current SEO
>>>>>>>> implementations
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > Can you suggest a book to read to better understand Lucenes
>>>>>>>> pattern dissection and match algorithms?
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > Many Thanks for helpful guidance
>>>>>>>> > Martin
>>>>>>>> > __
>>>>>>>> >
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > From: Erik Hatcher <erik.hatc...@gmail.com>
>>>>>>>> > Sent: Sunday, April 30, 2017 2:06 PM
>>>>>>>> >
>>>>>>>> > To: dev@lucene.apache.org
>>>>>>>> > Subject: Re: Release 6.6
>>>>>>>> >
>>>>>>>> > Martin -
>>>>>>>> >
>>>>>>>> > I have to admit to still being unsure what the gist is here - is
>>>>>>>> there a bug?   Apologies for not catching what you’re saying/showing 
>>>>>>>> here.
>>>>>>>> Again, as far as I can tell SpanPayloadCheckQuery is working as 
>>>>>>>> expected
>>>>>>>> now.
>>>>>>>> >
>>>>>>>> > I’m going to focus purely on SOLR-1485 this week to get it
>>>>>>>> committed for 6.6.  If there is an issue to address with your work 
>>>>>>>> would
>>>>>>>> you please open a JIRA and include your patch there?
>>>>>>>> >
>>>>>>>> > Thanks,
>>>>>>>> > Erik
>>>>>>>> >
>>>>>>>> >
>>>>

Re: Release 6.6

2017-06-05 Thread Erick Erickson
gt;> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi
>>>>>>>>> > <jim.feren...@gmail.com> wrote:
>>>>>>>>> > Hi,
>>>>>>>>> > Would be great to have
>>>>>>>>> > https://issues.apache.org/jira/browse/LUCENE-7824 included for 6.6. 
>>>>>>>>> > Can I
>>>>>>>>> > merge the fix this week end Ishan ?
>>>>>>>>> >
>>>>>>>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya
>>>>>>>>> > <ichattopadhy...@gmail.com>:
>>>>>>>>> > Done, Adrien. Thanks!
>>>>>>>>> >
>>>>>>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com>
>>>>>>>>> > wrote:
>>>>>>>>> > Ishan, wdyt about running addVersion on the branch_6x/master as
>>>>>>>>> > well? I think it would help realize that the 6.6 branch has been 
>>>>>>>>> > cut when
>>>>>>>>> > looking at the CHANGES.txt file and not forget about backporting?
>>>>>>>>> >
>>>>>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya
>>>>>>>>> > <ichattopadhy...@gmail.com> a écrit :
>>>>>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to
>>>>>>>>> > add bugfixes to the branch, if needed.
>>>>>>>>> > Planning to build the first RC on 15 May. Please let me know if
>>>>>>>>> > there are any objections.
>>>>>>>>> >
>>>>>>>>> > Thanks,
>>>>>>>>> > Ishan
>>>>>>>>> >
>>>>>>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya
>>>>>>>>> > <ichattopadhy...@gmail.com> wrote:
>>>>>>>>> > Planning to cut the release branch in another 10-12 hours.
>>>>>>>>> >
>>>>>>>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty
>>>>>>>>> > <mgai...@hotmail.com> wrote:
>>>>>>>>> > i was wondering if there was a specific test for
>>>>>>>>> > SpanPayloadCheckQuery method
>>>>>>>>> >
>>>>>>>>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > the only implementation i could locate was in collectLeaf of
>>>>>>>>> > SpanPayloadCheckQuery
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > I will submit JIRA with Patch
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > as a CS *dinosaur* I am more familiar with LISP for dissecting
>>>>>>>>> > sentence fragments (what we called phenomes) than current SEO
>>>>>>>>> > implementations
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > Can you suggest a book to read to better understand Lucenes
>>>>>>>>> > pattern dissection and match algorithms?
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > Many Thanks for helpful guidance
>>>>>>>>> > Martin
>>>>>>>>> > __
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> >
>>>>>>>>> > From: Erik Hatcher <erik.hatc...@gmail.com>
>>>>>>>>> > Sent: Sunday, April 30, 2017 2:06 PM
>>>>>>>>> >
>>>>>>>>> > To: dev@lucene.apache.org
>>>>>>>>> > Subject: Re: Release 6.6
>>>>>>>>> >
>>>>>>>>> > Martin -
>>>>>>>>> >
>>>>>>>>> > I have to admit to still being unsure what the gist is here - is
>>>>>>>>&g

Re: Release 6.6

2017-06-05 Thread Mike Drob
t;>>>> ichattopadhy...@gmail.com>:
>>>>>>>> > Done, Adrien. Thanks!
>>>>>>>> >
>>>>>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com>
>>>>>>>> wrote:
>>>>>>>> > Ishan, wdyt about running addVersion on the branch_6x/master as
>>>>>>>> well? I think it would help realize that the 6.6 branch has been cut 
>>>>>>>> when
>>>>>>>> looking at the CHANGES.txt file and not forget about backporting?
>>>>>>>> >
>>>>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>>>>>>> ichattopadhy...@gmail.com> a écrit :
>>>>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to
>>>>>>>> add bugfixes to the branch, if needed.
>>>>>>>> > Planning to build the first RC on 15 May. Please let me know if
>>>>>>>> there are any objections.
>>>>>>>> >
>>>>>>>> > Thanks,
>>>>>>>> > Ishan
>>>>>>>> >
>>>>>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>>>>>>> ichattopadhy...@gmail.com> wrote:
>>>>>>>> > Planning to cut the release branch in another 10-12 hours.
>>>>>>>> >
>>>>>>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <
>>>>>>>> mgai...@hotmail.com> wrote:
>>>>>>>> > i was wondering if there was a specific test for
>>>>>>>> SpanPayloadCheckQuery method
>>>>>>>> >
>>>>>>>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>>>>> >
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > the only implementation i could locate was in collectLeaf of
>>>>>>>> SpanPayloadCheckQuery
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > I will submit JIRA with Patch
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > as a CS *dinosaur* I am more familiar with LISP for dissecting
>>>>>>>> sentence fragments (what we called phenomes) than current SEO
>>>>>>>> implementations
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > Can you suggest a book to read to better understand Lucenes
>>>>>>>> pattern dissection and match algorithms?
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > Many Thanks for helpful guidance
>>>>>>>> > Martin
>>>>>>>> > __
>>>>>>>> >
>>>>>>>> >
>>>>>>>> >
>>>>>>>> > From: Erik Hatcher <erik.hatc...@gmail.com>
>>>>>>>> > Sent: Sunday, April 30, 2017 2:06 PM
>>>>>>>> >
>>>>>>>> > To: dev@lucene.apache.org
>>>>>>>> > Subject: Re: Release 6.6
>>>>>>>> >
>>>>>>>> > Martin -
>>>>>>>> >
>>>>>>>> > I have to admit to still being unsure what the gist is here - is
>>>>>>>> there a bug?   Apologies for not catching what you’re saying/showing 
>>>>>>>> here.
>>>>>>>> Again, as far as I can tell SpanPayloadCheckQuery is working as 
>>>>>>>> expected
>>>>>>>> now.
>>>>>>>> >
>>>>>>>> > I’m going to focus purely on SOLR-1485 this week to get it
>>>>>>>> committed for 6.6.  If there is an issue to address with your work 
>>>>>>>> would
>>>>>>>> you please open a JIRA and include your patch there?
>>>>>>>> >
>>>>>>>> > Thanks,
>>>>>>>> > Erik
>>>>>>>> >
>>>>>>>> >
>>>>>>>> >> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com>
>>&

Re: Release 6.6

2017-06-05 Thread Ishan Chattopadhyaya
t;>>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>>>>>> ichattopadhy...@gmail.com> a écrit :
>>>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>>>>>>> bugfixes to the branch, if needed.
>>>>>>> > Planning to build the first RC on 15 May. Please let me know if
>>>>>>> there are any objections.
>>>>>>> >
>>>>>>> > Thanks,
>>>>>>> > Ishan
>>>>>>> >
>>>>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>>>>>> ichattopadhy...@gmail.com> wrote:
>>>>>>> > Planning to cut the release branch in another 10-12 hours.
>>>>>>> >
>>>>>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>>>>>>> wrote:
>>>>>>> > i was wondering if there was a specific test for
>>>>>>> SpanPayloadCheckQuery method
>>>>>>> >
>>>>>>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>>>> >
>>>>>>> >
>>>>>>> >
>>>>>>> > the only implementation i could locate was in collectLeaf of
>>>>>>> SpanPayloadCheckQuery
>>>>>>> >
>>>>>>> >
>>>>>>> > I will submit JIRA with Patch
>>>>>>> >
>>>>>>> >
>>>>>>> > as a CS *dinosaur* I am more familiar with LISP for dissecting
>>>>>>> sentence fragments (what we called phenomes) than current SEO
>>>>>>> implementations
>>>>>>> >
>>>>>>> >
>>>>>>> > Can you suggest a book to read to better understand Lucenes
>>>>>>> pattern dissection and match algorithms?
>>>>>>> >
>>>>>>> >
>>>>>>> > Many Thanks for helpful guidance
>>>>>>> > Martin
>>>>>>> > __
>>>>>>> >
>>>>>>> >
>>>>>>> >
>>>>>>> > From: Erik Hatcher <erik.hatc...@gmail.com>
>>>>>>> > Sent: Sunday, April 30, 2017 2:06 PM
>>>>>>> >
>>>>>>> > To: dev@lucene.apache.org
>>>>>>> > Subject: Re: Release 6.6
>>>>>>> >
>>>>>>> > Martin -
>>>>>>> >
>>>>>>> > I have to admit to still being unsure what the gist is here - is
>>>>>>> there a bug?   Apologies for not catching what you’re saying/showing 
>>>>>>> here.
>>>>>>> Again, as far as I can tell SpanPayloadCheckQuery is working as expected
>>>>>>> now.
>>>>>>> >
>>>>>>> > I’m going to focus purely on SOLR-1485 this week to get it
>>>>>>> committed for 6.6.  If there is an issue to address with your work would
>>>>>>> you please open a JIRA and include your patch there?
>>>>>>> >
>>>>>>> > Thanks,
>>>>>>> > Erik
>>>>>>> >
>>>>>>> >
>>>>>>> >> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com>
>>>>>>> wrote:
>>>>>>> >>
>>>>>>> >> Mornin' Erik
>>>>>>> >>
>>>>>>> >> there is a collectLeaf  override in 
>>>>>>> >> org.apache.lucene.queries.payloads.TestPayloadSpans
>>>>>>> ..but its never called:
>>>>>>> >>
>>>>>>> >> static class VerifyingCollector implements SpanCollector {
>>>>>>> >> List payloads = new ArrayList<>();
>>>>>>> >> @Override
>>>>>>> >> public void collectLeaf(PostingsEnum postings, int position,
>>>>>>> Term term) throws IOException {
>>>>>>> >>  
>>>>>>> >>  }
>>>>>>> >> }
>>>>>>> >>
>>>>>>> >> the m

Re: Release 6.6

2017-05-16 Thread Adrien Grand
Sorry to hear that, make sure to take care of yourself first, the release
can wait! I just backported to branch_6_6.

Le mar. 16 mai 2017 à 19:22, Ishan Chattopadhyaya <ichattopadhy...@gmail.com>
a écrit :

> Hi Adrien,
> Please proceed, by all means, to backport to release branch. I'm unwell
> today, and I haven't been able to start the RC cutting today; planning to
> do so after another 5-6 hours' rest.
> Thanks,
> Ishan
>
> On Tue, May 16, 2017 at 10:06 PM, Adrien Grand <jpou...@gmail.com> wrote:
>
>> Hi Ishan,
>>
>> I don't think it is worth restarting RC build in case you already
>> started, but in case you haven't started yet,
>> https://issues.apache.org/jira/browse/LUCENE-7831 could be nice to have
>> in 6.6.
>>
>> Le mar. 16 mai 2017 à 03:55, Ishan Chattopadhyaya <
>> ichattopadhy...@gmail.com> a écrit :
>>
>>> I attempted to build an RC, but failed repeatedly before realizing it
>>> was due to, apart from test failures, LUCENE-7830 / LUCENE-6438. I've
>>> manually cleared up the dead symlinks now. I'll build the RC on Tuesday, as
>>> I'm about to wrap up the day's work.
>>>
>>> On Mon, May 15, 2017 at 10:12 PM, Ishan Chattopadhyaya <
>>> ichattopadhy...@gmail.com> wrote:
>>>
>>>> I wish to backport a fix from SOLR-8440 (last commit) to the release
>>>> branch. It affects only the feature introduced in SOLR-8440. Please let me
>>>> know if someone has any objections.
>>>>
>>>> Also, I'm planning to build the RC in another 3-4 hours. Please let me
>>>> know if there's something that someone is working on which needs to get in
>>>> before that.
>>>>
>>>> Thanks and regards,
>>>> Ishan
>>>>
>>>>
>>>> On Sun, May 14, 2017 at 5:02 PM, Ishan Chattopadhyaya <
>>>> ichattopadhy...@gmail.com> wrote:
>>>>
>>>>> Sure Steve! Thanks!
>>>>>
>>>>> On Sun, May 14, 2017 at 2:34 PM, Steve Rowe <sar...@gmail.com> wrote:
>>>>>
>>>>>> Ishan,
>>>>>>
>>>>>> Okay to include https://issues.apache.org/jira/browse/LUCENE-7821
>>>>>> for 6.6?
>>>>>>
>>>>>> --
>>>>>> Steve
>>>>>> www.lucidworks.com
>>>>>>
>>>>>> > On May 12, 2017, at 12:51 PM, jim ferenczi <jim.feren...@gmail.com>
>>>>>> wrote:
>>>>>> >
>>>>>> > Ok thanks Ishan.
>>>>>> >
>>>>>> > Le 12 mai 2017 18:44, "Ishan Chattopadhyaya" <
>>>>>> ichattopadhy...@gmail.com> a écrit :
>>>>>> > Sure, Jim. Please go ahead!
>>>>>> >
>>>>>> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <
>>>>>> jim.feren...@gmail.com> wrote:
>>>>>> > Hi,
>>>>>> > Would be great to have
>>>>>> https://issues.apache.org/jira/browse/LUCENE-7824 included for 6.6.
>>>>>> Can I merge the fix this week end Ishan ?
>>>>>> >
>>>>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <
>>>>>> ichattopadhy...@gmail.com>:
>>>>>> > Done, Adrien. Thanks!
>>>>>> >
>>>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com>
>>>>>> wrote:
>>>>>> > Ishan, wdyt about running addVersion on the branch_6x/master as
>>>>>> well? I think it would help realize that the 6.6 branch has been cut when
>>>>>> looking at the CHANGES.txt file and not forget about backporting?
>>>>>> >
>>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>>>>> ichattopadhy...@gmail.com> a écrit :
>>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>>>>>> bugfixes to the branch, if needed.
>>>>>> > Planning to build the first RC on 15 May. Please let me know if
>>>>>> there are any objections.
>>>>>> >
>>>>>> > Thanks,
>>>>>> > Ishan
>>>>>> >
>>>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>>>>> ichattopadhy...@gmail.com> wrote:
>>>>>> > Planning to cut the relea

Re: Release 6.6

2017-05-16 Thread Ishan Chattopadhyaya
Hi Adrien,
Please proceed, by all means, to backport to release branch. I'm unwell
today, and I haven't been able to start the RC cutting today; planning to
do so after another 5-6 hours' rest.
Thanks,
Ishan

On Tue, May 16, 2017 at 10:06 PM, Adrien Grand <jpou...@gmail.com> wrote:

> Hi Ishan,
>
> I don't think it is worth restarting RC build in case you already started,
> but in case you haven't started yet, https://issues.apache.org/
> jira/browse/LUCENE-7831 could be nice to have in 6.6.
>
> Le mar. 16 mai 2017 à 03:55, Ishan Chattopadhyaya <
> ichattopadhy...@gmail.com> a écrit :
>
>> I attempted to build an RC, but failed repeatedly before realizing it was
>> due to, apart from test failures, LUCENE-7830 / LUCENE-6438. I've manually
>> cleared up the dead symlinks now. I'll build the RC on Tuesday, as I'm
>> about to wrap up the day's work.
>>
>> On Mon, May 15, 2017 at 10:12 PM, Ishan Chattopadhyaya <
>> ichattopadhy...@gmail.com> wrote:
>>
>>> I wish to backport a fix from SOLR-8440 (last commit) to the release
>>> branch. It affects only the feature introduced in SOLR-8440. Please let me
>>> know if someone has any objections.
>>>
>>> Also, I'm planning to build the RC in another 3-4 hours. Please let me
>>> know if there's something that someone is working on which needs to get in
>>> before that.
>>>
>>> Thanks and regards,
>>> Ishan
>>>
>>>
>>> On Sun, May 14, 2017 at 5:02 PM, Ishan Chattopadhyaya <
>>> ichattopadhy...@gmail.com> wrote:
>>>
>>>> Sure Steve! Thanks!
>>>>
>>>> On Sun, May 14, 2017 at 2:34 PM, Steve Rowe <sar...@gmail.com> wrote:
>>>>
>>>>> Ishan,
>>>>>
>>>>> Okay to include https://issues.apache.org/jira/browse/LUCENE-7821 for
>>>>> 6.6?
>>>>>
>>>>> --
>>>>> Steve
>>>>> www.lucidworks.com
>>>>>
>>>>> > On May 12, 2017, at 12:51 PM, jim ferenczi <jim.feren...@gmail.com>
>>>>> wrote:
>>>>> >
>>>>> > Ok thanks Ishan.
>>>>> >
>>>>> > Le 12 mai 2017 18:44, "Ishan Chattopadhyaya" <
>>>>> ichattopadhy...@gmail.com> a écrit :
>>>>> > Sure, Jim. Please go ahead!
>>>>> >
>>>>> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <
>>>>> jim.feren...@gmail.com> wrote:
>>>>> > Hi,
>>>>> > Would be great to have https://issues.apache.org/
>>>>> jira/browse/LUCENE-7824 included for 6.6. Can I merge the fix this
>>>>> week end Ishan ?
>>>>> >
>>>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <
>>>>> ichattopadhy...@gmail.com>:
>>>>> > Done, Adrien. Thanks!
>>>>> >
>>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com>
>>>>> wrote:
>>>>> > Ishan, wdyt about running addVersion on the branch_6x/master as
>>>>> well? I think it would help realize that the 6.6 branch has been cut when
>>>>> looking at the CHANGES.txt file and not forget about backporting?
>>>>> >
>>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>>>> ichattopadhy...@gmail.com> a écrit :
>>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>>>>> bugfixes to the branch, if needed.
>>>>> > Planning to build the first RC on 15 May. Please let me know if
>>>>> there are any objections.
>>>>> >
>>>>> > Thanks,
>>>>> > Ishan
>>>>> >
>>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>>>> ichattopadhy...@gmail.com> wrote:
>>>>> > Planning to cut the release branch in another 10-12 hours.
>>>>> >
>>>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>>>>> wrote:
>>>>> > i was wondering if there was a specific test for
>>>>> SpanPayloadCheckQuery method
>>>>> >
>>>>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>> >
>>>>> >
>>>>> >
>>>>> > the only implementation i could locate was in collectLeaf of
>&

Re: Release 6.6

2017-05-16 Thread Adrien Grand
Hi Ishan,

I don't think it is worth restarting RC build in case you already started,
but in case you haven't started yet,
https://issues.apache.org/jira/browse/LUCENE-7831 could be nice to have in
6.6.

Le mar. 16 mai 2017 à 03:55, Ishan Chattopadhyaya <ichattopadhy...@gmail.com>
a écrit :

> I attempted to build an RC, but failed repeatedly before realizing it was
> due to, apart from test failures, LUCENE-7830 / LUCENE-6438. I've manually
> cleared up the dead symlinks now. I'll build the RC on Tuesday, as I'm
> about to wrap up the day's work.
>
> On Mon, May 15, 2017 at 10:12 PM, Ishan Chattopadhyaya <
> ichattopadhy...@gmail.com> wrote:
>
>> I wish to backport a fix from SOLR-8440 (last commit) to the release
>> branch. It affects only the feature introduced in SOLR-8440. Please let me
>> know if someone has any objections.
>>
>> Also, I'm planning to build the RC in another 3-4 hours. Please let me
>> know if there's something that someone is working on which needs to get in
>> before that.
>>
>> Thanks and regards,
>> Ishan
>>
>>
>> On Sun, May 14, 2017 at 5:02 PM, Ishan Chattopadhyaya <
>> ichattopadhy...@gmail.com> wrote:
>>
>>> Sure Steve! Thanks!
>>>
>>> On Sun, May 14, 2017 at 2:34 PM, Steve Rowe <sar...@gmail.com> wrote:
>>>
>>>> Ishan,
>>>>
>>>> Okay to include https://issues.apache.org/jira/browse/LUCENE-7821 for
>>>> 6.6?
>>>>
>>>> --
>>>> Steve
>>>> www.lucidworks.com
>>>>
>>>> > On May 12, 2017, at 12:51 PM, jim ferenczi <jim.feren...@gmail.com>
>>>> wrote:
>>>> >
>>>> > Ok thanks Ishan.
>>>> >
>>>> > Le 12 mai 2017 18:44, "Ishan Chattopadhyaya" <
>>>> ichattopadhy...@gmail.com> a écrit :
>>>> > Sure, Jim. Please go ahead!
>>>> >
>>>> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <
>>>> jim.feren...@gmail.com> wrote:
>>>> > Hi,
>>>> > Would be great to have
>>>> https://issues.apache.org/jira/browse/LUCENE-7824 included for 6.6.
>>>> Can I merge the fix this week end Ishan ?
>>>> >
>>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <
>>>> ichattopadhy...@gmail.com>:
>>>> > Done, Adrien. Thanks!
>>>> >
>>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com>
>>>> wrote:
>>>> > Ishan, wdyt about running addVersion on the branch_6x/master as well?
>>>> I think it would help realize that the 6.6 branch has been cut when looking
>>>> at the CHANGES.txt file and not forget about backporting?
>>>> >
>>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>>> ichattopadhy...@gmail.com> a écrit :
>>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>>>> bugfixes to the branch, if needed.
>>>> > Planning to build the first RC on 15 May. Please let me know if there
>>>> are any objections.
>>>> >
>>>> > Thanks,
>>>> > Ishan
>>>> >
>>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>>> ichattopadhy...@gmail.com> wrote:
>>>> > Planning to cut the release branch in another 10-12 hours.
>>>> >
>>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>>>> wrote:
>>>> > i was wondering if there was a specific test for
>>>> SpanPayloadCheckQuery method
>>>> >
>>>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>> >
>>>> >
>>>> >
>>>> > the only implementation i could locate was in collectLeaf of
>>>> SpanPayloadCheckQuery
>>>> >
>>>> >
>>>> > I will submit JIRA with Patch
>>>> >
>>>> >
>>>> > as a CS *dinosaur* I am more familiar with LISP for dissecting
>>>> sentence fragments (what we called phenomes) than current SEO
>>>> implementations
>>>> >
>>>> >
>>>> > Can you suggest a book to read to better understand Lucenes pattern
>>>> dissection and match algorithms?
>>>> >
>>>> >
>>>> > Many Thanks for helpful guidance
>>

Re: Release 6.6

2017-05-15 Thread Ishan Chattopadhyaya
I attempted to build an RC, but failed repeatedly before realizing it was
due to, apart from test failures, LUCENE-7830 / LUCENE-6438. I've manually
cleared up the dead symlinks now. I'll build the RC on Tuesday, as I'm
about to wrap up the day's work.

On Mon, May 15, 2017 at 10:12 PM, Ishan Chattopadhyaya <
ichattopadhy...@gmail.com> wrote:

> I wish to backport a fix from SOLR-8440 (last commit) to the release
> branch. It affects only the feature introduced in SOLR-8440. Please let me
> know if someone has any objections.
>
> Also, I'm planning to build the RC in another 3-4 hours. Please let me
> know if there's something that someone is working on which needs to get in
> before that.
>
> Thanks and regards,
> Ishan
>
>
> On Sun, May 14, 2017 at 5:02 PM, Ishan Chattopadhyaya <
> ichattopadhy...@gmail.com> wrote:
>
>> Sure Steve! Thanks!
>>
>> On Sun, May 14, 2017 at 2:34 PM, Steve Rowe <sar...@gmail.com> wrote:
>>
>>> Ishan,
>>>
>>> Okay to include https://issues.apache.org/jira/browse/LUCENE-7821 for
>>> 6.6?
>>>
>>> --
>>> Steve
>>> www.lucidworks.com
>>>
>>> > On May 12, 2017, at 12:51 PM, jim ferenczi <jim.feren...@gmail.com>
>>> wrote:
>>> >
>>> > Ok thanks Ishan.
>>> >
>>> > Le 12 mai 2017 18:44, "Ishan Chattopadhyaya" <
>>> ichattopadhy...@gmail.com> a écrit :
>>> > Sure, Jim. Please go ahead!
>>> >
>>> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <jim.feren...@gmail.com>
>>> wrote:
>>> > Hi,
>>> > Would be great to have https://issues.apache.org/jira
>>> /browse/LUCENE-7824 included for 6.6. Can I merge the fix this week end
>>> Ishan ?
>>> >
>>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <
>>> ichattopadhy...@gmail.com>:
>>> > Done, Adrien. Thanks!
>>> >
>>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com>
>>> wrote:
>>> > Ishan, wdyt about running addVersion on the branch_6x/master as well?
>>> I think it would help realize that the 6.6 branch has been cut when looking
>>> at the CHANGES.txt file and not forget about backporting?
>>> >
>>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>> ichattopadhy...@gmail.com> a écrit :
>>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>>> bugfixes to the branch, if needed.
>>> > Planning to build the first RC on 15 May. Please let me know if there
>>> are any objections.
>>> >
>>> > Thanks,
>>> > Ishan
>>> >
>>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>> ichattopadhy...@gmail.com> wrote:
>>> > Planning to cut the release branch in another 10-12 hours.
>>> >
>>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>>> wrote:
>>> > i was wondering if there was a specific test for SpanPayloadCheckQuery
>>> method
>>> >
>>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>>> >
>>> >
>>> >
>>> > the only implementation i could locate was in collectLeaf of
>>> SpanPayloadCheckQuery
>>> >
>>> >
>>> > I will submit JIRA with Patch
>>> >
>>> >
>>> > as a CS *dinosaur* I am more familiar with LISP for dissecting
>>> sentence fragments (what we called phenomes) than current SEO
>>> implementations
>>> >
>>> >
>>> > Can you suggest a book to read to better understand Lucenes pattern
>>> dissection and match algorithms?
>>> >
>>> >
>>> > Many Thanks for helpful guidance
>>> > Martin
>>> > __
>>> >
>>> >
>>> >
>>> > From: Erik Hatcher <erik.hatc...@gmail.com>
>>> > Sent: Sunday, April 30, 2017 2:06 PM
>>> >
>>> > To: dev@lucene.apache.org
>>> > Subject: Re: Release 6.6
>>> >
>>> > Martin -
>>> >
>>> > I have to admit to still being unsure what the gist is here - is there
>>> a bug?   Apologies for not catching what you’re saying/showing here.
>>> Again, as far as I can tell SpanPayloadCheckQuery is working as expected
>>> now.
>>> >
>>> > I’m 

Re: Release 6.6

2017-05-15 Thread Ishan Chattopadhyaya
I wish to backport a fix from SOLR-8440 (last commit) to the release
branch. It affects only the feature introduced in SOLR-8440. Please let me
know if someone has any objections.

Also, I'm planning to build the RC in another 3-4 hours. Please let me know
if there's something that someone is working on which needs to get in
before that.

Thanks and regards,
Ishan


On Sun, May 14, 2017 at 5:02 PM, Ishan Chattopadhyaya <
ichattopadhy...@gmail.com> wrote:

> Sure Steve! Thanks!
>
> On Sun, May 14, 2017 at 2:34 PM, Steve Rowe <sar...@gmail.com> wrote:
>
>> Ishan,
>>
>> Okay to include https://issues.apache.org/jira/browse/LUCENE-7821 for
>> 6.6?
>>
>> --
>> Steve
>> www.lucidworks.com
>>
>> > On May 12, 2017, at 12:51 PM, jim ferenczi <jim.feren...@gmail.com>
>> wrote:
>> >
>> > Ok thanks Ishan.
>> >
>> > Le 12 mai 2017 18:44, "Ishan Chattopadhyaya" <ichattopadhy...@gmail.com>
>> a écrit :
>> > Sure, Jim. Please go ahead!
>> >
>> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <jim.feren...@gmail.com>
>> wrote:
>> > Hi,
>> > Would be great to have https://issues.apache.org/jira
>> /browse/LUCENE-7824 included for 6.6. Can I merge the fix this week end
>> Ishan ?
>> >
>> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <
>> ichattopadhy...@gmail.com>:
>> > Done, Adrien. Thanks!
>> >
>> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com>
>> wrote:
>> > Ishan, wdyt about running addVersion on the branch_6x/master as well? I
>> think it would help realize that the 6.6 branch has been cut when looking
>> at the CHANGES.txt file and not forget about backporting?
>> >
>> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>> ichattopadhy...@gmail.com> a écrit :
>> > I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>> bugfixes to the branch, if needed.
>> > Planning to build the first RC on 15 May. Please let me know if there
>> are any objections.
>> >
>> > Thanks,
>> > Ishan
>> >
>> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>> ichattopadhy...@gmail.com> wrote:
>> > Planning to cut the release branch in another 10-12 hours.
>> >
>> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>> wrote:
>> > i was wondering if there was a specific test for SpanPayloadCheckQuery
>> method
>> >
>> > matches = payloadToMatch.get(upto).bytesEquals(payload);
>> >
>> >
>> >
>> > the only implementation i could locate was in collectLeaf of
>> SpanPayloadCheckQuery
>> >
>> >
>> > I will submit JIRA with Patch
>> >
>> >
>> > as a CS *dinosaur* I am more familiar with LISP for dissecting sentence
>> fragments (what we called phenomes) than current SEO implementations
>> >
>> >
>> > Can you suggest a book to read to better understand Lucenes pattern
>> dissection and match algorithms?
>> >
>> >
>> > Many Thanks for helpful guidance
>> > Martin
>> > __
>> >
>> >
>> >
>> > From: Erik Hatcher <erik.hatc...@gmail.com>
>> > Sent: Sunday, April 30, 2017 2:06 PM
>> >
>> > To: dev@lucene.apache.org
>> > Subject: Re: Release 6.6
>> >
>> > Martin -
>> >
>> > I have to admit to still being unsure what the gist is here - is there
>> a bug?   Apologies for not catching what you’re saying/showing here.
>> Again, as far as I can tell SpanPayloadCheckQuery is working as expected
>> now.
>> >
>> > I’m going to focus purely on SOLR-1485 this week to get it committed
>> for 6.6.  If there is an issue to address with your work would you please
>> open a JIRA and include your patch there?
>> >
>> > Thanks,
>> > Erik
>> >
>> >
>> >> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com>
>> wrote:
>> >>
>> >> Mornin' Erik
>> >>
>> >> there is a collectLeaf  override in 
>> >> org.apache.lucene.queries.payloads.TestPayloadSpans
>> ..but its never called:
>> >>
>> >> static class VerifyingCollector implements SpanCollector {
>> >> List payloads = new ArrayList<>();
>> >> @Override
>> >> public voi

Re: Release 6.6

2017-05-14 Thread Ishan Chattopadhyaya
Sure Steve! Thanks!

On Sun, May 14, 2017 at 2:34 PM, Steve Rowe <sar...@gmail.com> wrote:

> Ishan,
>
> Okay to include https://issues.apache.org/jira/browse/LUCENE-7821 for 6.6?
>
> --
> Steve
> www.lucidworks.com
>
> > On May 12, 2017, at 12:51 PM, jim ferenczi <jim.feren...@gmail.com>
> wrote:
> >
> > Ok thanks Ishan.
> >
> > Le 12 mai 2017 18:44, "Ishan Chattopadhyaya" <ichattopadhy...@gmail.com>
> a écrit :
> > Sure, Jim. Please go ahead!
> >
> > On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <jim.feren...@gmail.com>
> wrote:
> > Hi,
> > Would be great to have https://issues.apache.org/jira/browse/LUCENE-7824
> included for 6.6. Can I merge the fix this week end Ishan ?
> >
> > 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <
> ichattopadhy...@gmail.com>:
> > Done, Adrien. Thanks!
> >
> > On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com> wrote:
> > Ishan, wdyt about running addVersion on the branch_6x/master as well? I
> think it would help realize that the 6.6 branch has been cut when looking
> at the CHANGES.txt file and not forget about backporting?
> >
> > Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
> ichattopadhy...@gmail.com> a écrit :
> > I've cut the branch for 6.6. (branch_6_6). Please feel free to add
> bugfixes to the branch, if needed.
> > Planning to build the first RC on 15 May. Please let me know if there
> are any objections.
> >
> > Thanks,
> > Ishan
> >
> > On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
> ichattopadhy...@gmail.com> wrote:
> > Planning to cut the release branch in another 10-12 hours.
> >
> > On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
> wrote:
> > i was wondering if there was a specific test for SpanPayloadCheckQuery
> method
> >
> > matches = payloadToMatch.get(upto).bytesEquals(payload);
> >
> >
> >
> > the only implementation i could locate was in collectLeaf of
> SpanPayloadCheckQuery
> >
> >
> > I will submit JIRA with Patch
> >
> >
> > as a CS *dinosaur* I am more familiar with LISP for dissecting sentence
> fragments (what we called phenomes) than current SEO implementations
> >
> >
> > Can you suggest a book to read to better understand Lucenes pattern
> dissection and match algorithms?
> >
> >
> > Many Thanks for helpful guidance
> > Martin
> > __
> >
> >
> >
> > From: Erik Hatcher <erik.hatc...@gmail.com>
> > Sent: Sunday, April 30, 2017 2:06 PM
> >
> > To: dev@lucene.apache.org
> > Subject: Re: Release 6.6
> >
> > Martin -
> >
> > I have to admit to still being unsure what the gist is here - is there a
> bug?   Apologies for not catching what you’re saying/showing here.  Again,
> as far as I can tell SpanPayloadCheckQuery is working as expected now.
> >
> > I’m going to focus purely on SOLR-1485 this week to get it committed for
> 6.6.  If there is an issue to address with your work would you please open
> a JIRA and include your patch there?
> >
> > Thanks,
> > Erik
> >
> >
> >> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com> wrote:
> >>
> >> Mornin' Erik
> >>
> >> there is a collectLeaf  override in 
> >> org.apache.lucene.queries.payloads.TestPayloadSpans
> ..but its never called:
> >>
> >> static class VerifyingCollector implements SpanCollector {
> >> List payloads = new ArrayList<>();
> >> @Override
> >> public void collectLeaf(PostingsEnum postings, int position, Term
> term) throws IOException {
> >>  
> >>  }
> >> }
> >>
> >> the modification in 
> >> org.apache.lucene.queries.payloads.TestPayloadCheckQuery
> tests collectLeaf for query
> >>
> >> //initialise term
> >> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 231 before
> term1=new org.apache.lucene.index.Term('field','withPayload')");
> >> org.apache.lucene.index.Term term1=new 
> >> org.apache.lucene.index.Term("field",
> "withPayload");
> >> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 233
> term1="+term1);
> >>
> >> //assume position is 5
> >> int position=5;
> >> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 235
> positio

Re: Release 6.6

2017-05-14 Thread Uwe Schindler
Ok from my side! Uwe

Am 14. Mai 2017 11:04:33 MESZ schrieb Steve Rowe <sar...@gmail.com>:
>Ishan,
>
>Okay to include https://issues.apache.org/jira/browse/LUCENE-7821 for
>6.6?
>
>--
>Steve
>www.lucidworks.com
>
>> On May 12, 2017, at 12:51 PM, jim ferenczi <jim.feren...@gmail.com>
>wrote:
>> 
>> Ok thanks Ishan.
>> 
>> Le 12 mai 2017 18:44, "Ishan Chattopadhyaya"
><ichattopadhy...@gmail.com> a écrit :
>> Sure, Jim. Please go ahead!
>> 
>> On Fri, May 12, 2017 at 10:01 PM, jim ferenczi
><jim.feren...@gmail.com> wrote:
>> Hi,
>> Would be great to have
>https://issues.apache.org/jira/browse/LUCENE-7824 included for 6.6. Can
>I merge the fix this week end Ishan ?
>> 
>> 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya
><ichattopadhy...@gmail.com>:
>> Done, Adrien. Thanks!
>> 
>> On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com>
>wrote:
>> Ishan, wdyt about running addVersion on the branch_6x/master as well?
>I think it would help realize that the 6.6 branch has been cut when
>looking at the CHANGES.txt file and not forget about backporting?
>> 
>> Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya
><ichattopadhy...@gmail.com> a écrit :
>> I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>bugfixes to the branch, if needed.
>> Planning to build the first RC on 15 May. Please let me know if there
>are any objections.
>> 
>> Thanks,
>> Ishan
>> 
>> On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya
><ichattopadhy...@gmail.com> wrote:
>> Planning to cut the release branch in another 10-12 hours.
>> 
>> On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>wrote:
>> i was wondering if there was a specific test for
>SpanPayloadCheckQuery method
>> 
>> matches = payloadToMatch.get(upto).bytesEquals(payload);
>> 
>> 
>> 
>> the only implementation i could locate was in collectLeaf of
>SpanPayloadCheckQuery
>> 
>> 
>> I will submit JIRA with Patch
>> 
>> 
>> as a CS *dinosaur* I am more familiar with LISP for dissecting
>sentence fragments (what we called phenomes) than current SEO
>implementations
>> 
>> 
>> Can you suggest a book to read to better understand Lucenes pattern
>dissection and match algorithms?
>> 
>> 
>> Many Thanks for helpful guidance
>> Martin  
>> __ 
>> 
>> 
>> 
>> From: Erik Hatcher <erik.hatc...@gmail.com>
>> Sent: Sunday, April 30, 2017 2:06 PM
>> 
>> To: dev@lucene.apache.org
>> Subject: Re: Release 6.6
>>  
>> Martin -
>> 
>> I have to admit to still being unsure what the gist is here - is
>there a bug?   Apologies for not catching what you’re saying/showing
>here.  Again, as far as I can tell SpanPayloadCheckQuery is working as
>expected now.  
>> 
>> I’m going to focus purely on SOLR-1485 this week to get it committed
>for 6.6.  If there is an issue to address with your work would you
>please open a JIRA and include your patch there?
>> 
>> Thanks,
>> Erik
>> 
>> 
>>> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com>
>wrote:
>>> 
>>> Mornin' Erik
>>> 
>>> there is a collectLeaf  override in
>org.apache.lucene.queries.payloads.TestPayloadSpans ..but its never
>called:
>>> 
>>> static class VerifyingCollector implements SpanCollector {
>>> List payloads = new ArrayList<>();
>>> @Override
>>> public void collectLeaf(PostingsEnum postings, int position,
>Term term) throws IOException {
>>>  
>>>  }
>>> }
>>> 
>>> the modification in
>org.apache.lucene.queries.payloads.TestPayloadCheckQuery tests
>collectLeaf for query
>>> 
>>> //initialise term
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 231
>before term1=new org.apache.lucene.index.Term('field','withPayload')");
>>> org.apache.lucene.index.Term term1=new
>org.apache.lucene.index.Term("field", "withPayload");
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 233
>term1="+term1);
>>> 
>>> //assume position is 5
>>> int position=5;
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 235
>position="+position);
>>> 
>>> BytesRef pay = new BytesRef("pos: " + position

Re: Release 6.6

2017-05-14 Thread Steve Rowe
Ishan,

Okay to include https://issues.apache.org/jira/browse/LUCENE-7821 for 6.6?

--
Steve
www.lucidworks.com

> On May 12, 2017, at 12:51 PM, jim ferenczi <jim.feren...@gmail.com> wrote:
> 
> Ok thanks Ishan.
> 
> Le 12 mai 2017 18:44, "Ishan Chattopadhyaya" <ichattopadhy...@gmail.com> a 
> écrit :
> Sure, Jim. Please go ahead!
> 
> On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <jim.feren...@gmail.com> wrote:
> Hi,
> Would be great to have https://issues.apache.org/jira/browse/LUCENE-7824 
> included for 6.6. Can I merge the fix this week end Ishan ?
> 
> 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <ichattopadhy...@gmail.com>:
> Done, Adrien. Thanks!
> 
> On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com> wrote:
> Ishan, wdyt about running addVersion on the branch_6x/master as well? I think 
> it would help realize that the 6.6 branch has been cut when looking at the 
> CHANGES.txt file and not forget about backporting?
> 
> Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <ichattopadhy...@gmail.com> 
> a écrit :
> I've cut the branch for 6.6. (branch_6_6). Please feel free to add bugfixes 
> to the branch, if needed.
> Planning to build the first RC on 15 May. Please let me know if there are any 
> objections.
> 
> Thanks,
> Ishan
> 
> On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya 
> <ichattopadhy...@gmail.com> wrote:
> Planning to cut the release branch in another 10-12 hours.
> 
> On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com> wrote:
> i was wondering if there was a specific test for SpanPayloadCheckQuery method
> 
> matches = payloadToMatch.get(upto).bytesEquals(payload);
> 
> 
> 
> the only implementation i could locate was in collectLeaf of 
> SpanPayloadCheckQuery
> 
> 
> I will submit JIRA with Patch
> 
> 
> as a CS *dinosaur* I am more familiar with LISP for dissecting sentence 
> fragments (what we called phenomes) than current SEO implementations
> 
> 
> Can you suggest a book to read to better understand Lucenes pattern 
> dissection and match algorithms?
> 
> 
> Many Thanks for helpful guidance
> Martin  
> __ 
> 
> 
> 
> From: Erik Hatcher <erik.hatc...@gmail.com>
> Sent: Sunday, April 30, 2017 2:06 PM
> 
> To: dev@lucene.apache.org
> Subject: Re: Release 6.6
>  
> Martin -
> 
> I have to admit to still being unsure what the gist is here - is there a bug? 
>   Apologies for not catching what you’re saying/showing here.  Again, as far 
> as I can tell SpanPayloadCheckQuery is working as expected now.  
> 
> I’m going to focus purely on SOLR-1485 this week to get it committed for 6.6. 
>  If there is an issue to address with your work would you please open a JIRA 
> and include your patch there?
> 
> Thanks,
> Erik
> 
> 
>> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com> wrote:
>> 
>> Mornin' Erik
>> 
>> there is a collectLeaf  override in 
>> org.apache.lucene.queries.payloads.TestPayloadSpans ..but its never called:
>> 
>> static class VerifyingCollector implements SpanCollector {
>> List payloads = new ArrayList<>();
>> @Override
>> public void collectLeaf(PostingsEnum postings, int position, Term term) 
>> throws IOException {
>>  
>>  }
>> }
>> 
>> the modification in org.apache.lucene.queries.payloads.TestPayloadCheckQuery 
>> tests collectLeaf for query
>> 
>> //initialise term
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 231 before 
>> term1=new org.apache.lucene.index.Term('field','withPayload')");
>> org.apache.lucene.index.Term term1=new org.apache.lucene.index.Term("field", 
>> "withPayload");
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 233 
>> term1="+term1);
>> 
>> //assume position is 5
>> int position=5;
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 235 
>> position="+position);
>> 
>> BytesRef pay = new BytesRef("pos: " + position);
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 237 
>> pay="+pay);
>> 
>> //build spanQuery with term parameter
>> org.apache.lucene.search.spans.SpanQuery spanQuery1 = new 
>> SpanTermQuery(term1);
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 239 
>> spanQuery1="+spanQuery1);
>> 
>> //add BytesRef to payloadT

Re: Release 6.6

2017-05-12 Thread jim ferenczi
Ok thanks Ishan.

Le 12 mai 2017 18:44, "Ishan Chattopadhyaya" <ichattopadhy...@gmail.com> a
écrit :

Sure, Jim. Please go ahead!

On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <jim.feren...@gmail.com>
wrote:

> Hi,
> Would be great to have https://issues.apache.org/jira/browse/LUCENE-7824
> included for 6.6. Can I merge the fix this week end Ishan ?
>
> 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <ichattopadhy...@gmail.com
> >:
>
>> Done, Adrien. Thanks!
>>
>> On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com> wrote:
>>
>>> Ishan, wdyt about running addVersion on the branch_6x/master as well? I
>>> think it would help realize that the 6.6 branch has been cut when looking
>>> at the CHANGES.txt file and not forget about backporting?
>>>
>>> Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>> ichattopadhy...@gmail.com> a écrit :
>>>
>>>> I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>>>> bugfixes to the branch, if needed.
>>>> Planning to build the first RC on 15 May. Please let me know if there
>>>> are any objections.
>>>>
>>>> Thanks,
>>>> Ishan
>>>>
>>>> On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>>> ichattopadhy...@gmail.com> wrote:
>>>>
>>>>> Planning to cut the release branch in another 10-12 hours.
>>>>>
>>>>> On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>>>>> wrote:
>>>>>
>>>>>> i was wondering if there was a specific test for
>>>>>> SpanPayloadCheckQuery method
>>>>>>
>>>>>> matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>>>
>>>>>>
>>>>>> the only implementation i could locate was in collectLeaf of
>>>>>> SpanPayloadCheckQuery
>>>>>>
>>>>>>
>>>>>> I will submit JIRA with Patch
>>>>>>
>>>>>>
>>>>>> as a CS *dinosaur* I am more familiar with LISP for dissecting
>>>>>> sentence fragments (what we called phenomes) than current SEO
>>>>>> implementations
>>>>>>
>>>>>>
>>>>>> Can you suggest a book to read to better understand Lucenes pattern
>>>>>> dissection and match algorithms?
>>>>>>
>>>>>>
>>>>>> Many Thanks for helpful guidance
>>>>>> Martin
>>>>>> __
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> *From:* Erik Hatcher <erik.hatc...@gmail.com>
>>>>>> *Sent:* Sunday, April 30, 2017 2:06 PM
>>>>>>
>>>>>> *To:* dev@lucene.apache.org
>>>>>> *Subject:* Re: Release 6.6
>>>>>>
>>>>>> Martin -
>>>>>>
>>>>>> I have to admit to still being unsure what the gist is here - is
>>>>>> there a bug?   Apologies for not catching what you’re saying/showing 
>>>>>> here.
>>>>>> Again, as far as I can tell SpanPayloadCheckQuery is working as expected
>>>>>> now.
>>>>>>
>>>>>> I’m going to focus purely on SOLR-1485 this week to get it committed
>>>>>> for 6.6.  If there is an issue to address with your work would you please
>>>>>> open a JIRA and include your patch there?
>>>>>>
>>>>>> Thanks,
>>>>>> Erik
>>>>>>
>>>>>>
>>>>>> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com>
>>>>>> wrote:
>>>>>>
>>>>>> Mornin' Erik
>>>>>>
>>>>>> there is a collectLeaf  override in org.apache.lucene
>>>>>> .queries.payloads.TestPayloadSpans ..but its never called:
>>>>>>
>>>>>> static class VerifyingCollector implements SpanCollector {
>>>>>> List payloads = new ArrayList<>();
>>>>>> @Override
>>>>>> public void collectLeaf(PostingsEnum postings, int position, Term
>>>>>> term) throws 

Re: Release 6.6

2017-05-12 Thread Ishan Chattopadhyaya
Sure, Jim. Please go ahead!

On Fri, May 12, 2017 at 10:01 PM, jim ferenczi <jim.feren...@gmail.com>
wrote:

> Hi,
> Would be great to have https://issues.apache.org/jira/browse/LUCENE-7824
> included for 6.6. Can I merge the fix this week end Ishan ?
>
> 2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <ichattopadhy...@gmail.com
> >:
>
>> Done, Adrien. Thanks!
>>
>> On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com> wrote:
>>
>>> Ishan, wdyt about running addVersion on the branch_6x/master as well? I
>>> think it would help realize that the 6.6 branch has been cut when looking
>>> at the CHANGES.txt file and not forget about backporting?
>>>
>>> Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>>> ichattopadhy...@gmail.com> a écrit :
>>>
>>>> I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>>>> bugfixes to the branch, if needed.
>>>> Planning to build the first RC on 15 May. Please let me know if there
>>>> are any objections.
>>>>
>>>> Thanks,
>>>> Ishan
>>>>
>>>> On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>>> ichattopadhy...@gmail.com> wrote:
>>>>
>>>>> Planning to cut the release branch in another 10-12 hours.
>>>>>
>>>>> On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>>>>> wrote:
>>>>>
>>>>>> i was wondering if there was a specific test for
>>>>>> SpanPayloadCheckQuery method
>>>>>>
>>>>>> matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>>>
>>>>>>
>>>>>> the only implementation i could locate was in collectLeaf of
>>>>>> SpanPayloadCheckQuery
>>>>>>
>>>>>>
>>>>>> I will submit JIRA with Patch
>>>>>>
>>>>>>
>>>>>> as a CS *dinosaur* I am more familiar with LISP for dissecting
>>>>>> sentence fragments (what we called phenomes) than current SEO
>>>>>> implementations
>>>>>>
>>>>>>
>>>>>> Can you suggest a book to read to better understand Lucenes pattern
>>>>>> dissection and match algorithms?
>>>>>>
>>>>>>
>>>>>> Many Thanks for helpful guidance
>>>>>> Martin
>>>>>> __
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> *From:* Erik Hatcher <erik.hatc...@gmail.com>
>>>>>> *Sent:* Sunday, April 30, 2017 2:06 PM
>>>>>>
>>>>>> *To:* dev@lucene.apache.org
>>>>>> *Subject:* Re: Release 6.6
>>>>>>
>>>>>> Martin -
>>>>>>
>>>>>> I have to admit to still being unsure what the gist is here - is
>>>>>> there a bug?   Apologies for not catching what you’re saying/showing 
>>>>>> here.
>>>>>> Again, as far as I can tell SpanPayloadCheckQuery is working as expected
>>>>>> now.
>>>>>>
>>>>>> I’m going to focus purely on SOLR-1485 this week to get it committed
>>>>>> for 6.6.  If there is an issue to address with your work would you please
>>>>>> open a JIRA and include your patch there?
>>>>>>
>>>>>> Thanks,
>>>>>> Erik
>>>>>>
>>>>>>
>>>>>> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com>
>>>>>> wrote:
>>>>>>
>>>>>> Mornin' Erik
>>>>>>
>>>>>> there is a collectLeaf  override in org.apache.lucene
>>>>>> .queries.payloads.TestPayloadSpans ..but its never called:
>>>>>>
>>>>>> static class VerifyingCollector implements SpanCollector {
>>>>>> List payloads = new ArrayList<>();
>>>>>> @Override
>>>>>> public void collectLeaf(PostingsEnum postings, int position, Term
>>>>>> term) throws IOException {
>>>>>>  
>>>>>>  }
>>>>>> }
>>>>>>
>>>>>

Re: Release 6.6

2017-05-12 Thread jim ferenczi
Hi,
Would be great to have https://issues.apache.org/jira/browse/LUCENE-7824
included for 6.6. Can I merge the fix this week end Ishan ?

2017-05-11 16:45 GMT+02:00 Ishan Chattopadhyaya <ichattopadhy...@gmail.com>:

> Done, Adrien. Thanks!
>
> On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com> wrote:
>
>> Ishan, wdyt about running addVersion on the branch_6x/master as well? I
>> think it would help realize that the 6.6 branch has been cut when looking
>> at the CHANGES.txt file and not forget about backporting?
>>
>> Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
>> ichattopadhy...@gmail.com> a écrit :
>>
>>> I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>>> bugfixes to the branch, if needed.
>>> Planning to build the first RC on 15 May. Please let me know if there
>>> are any objections.
>>>
>>> Thanks,
>>> Ishan
>>>
>>> On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>>> ichattopadhy...@gmail.com> wrote:
>>>
>>>> Planning to cut the release branch in another 10-12 hours.
>>>>
>>>> On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>>>> wrote:
>>>>
>>>>> i was wondering if there was a specific test for SpanPayloadCheckQuery
>>>>> method
>>>>>
>>>>> matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>>
>>>>>
>>>>> the only implementation i could locate was in collectLeaf of
>>>>> SpanPayloadCheckQuery
>>>>>
>>>>>
>>>>> I will submit JIRA with Patch
>>>>>
>>>>>
>>>>> as a CS *dinosaur* I am more familiar with LISP for dissecting
>>>>> sentence fragments (what we called phenomes) than current SEO
>>>>> implementations
>>>>>
>>>>>
>>>>> Can you suggest a book to read to better understand Lucenes pattern
>>>>> dissection and match algorithms?
>>>>>
>>>>>
>>>>> Many Thanks for helpful guidance
>>>>> Martin
>>>>> __
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> *From:* Erik Hatcher <erik.hatc...@gmail.com>
>>>>> *Sent:* Sunday, April 30, 2017 2:06 PM
>>>>>
>>>>> *To:* dev@lucene.apache.org
>>>>> *Subject:* Re: Release 6.6
>>>>>
>>>>> Martin -
>>>>>
>>>>> I have to admit to still being unsure what the gist is here - is there
>>>>> a bug?   Apologies for not catching what you’re saying/showing here.
>>>>> Again, as far as I can tell SpanPayloadCheckQuery is working as expected
>>>>> now.
>>>>>
>>>>> I’m going to focus purely on SOLR-1485 this week to get it committed
>>>>> for 6.6.  If there is an issue to address with your work would you please
>>>>> open a JIRA and include your patch there?
>>>>>
>>>>> Thanks,
>>>>> Erik
>>>>>
>>>>>
>>>>> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com>
>>>>> wrote:
>>>>>
>>>>> Mornin' Erik
>>>>>
>>>>> there is a collectLeaf  override in org.apache.lucene
>>>>> .queries.payloads.TestPayloadSpans ..but its never called:
>>>>>
>>>>> static class VerifyingCollector implements SpanCollector {
>>>>> List payloads = new ArrayList<>();
>>>>> @Override
>>>>> public void collectLeaf(PostingsEnum postings, int position, Term
>>>>> term) throws IOException {
>>>>>  
>>>>>  }
>>>>> }
>>>>>
>>>>>
>>>>> the modification in org.apache.lucene.queries.payl
>>>>> oads.TestPayloadCheckQuery tests collectLeaf for query
>>>>>
>>>>> //initialise term
>>>>>
>>>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 231
>>>>> before term1=new org.apache.lucene.index.Term('
>>>>> field','withPayload')");
>>>>> org.apache.lucene.index.Term term1=new 
>>>>> org.apache.lucene.index.T

Re: Release 6.6

2017-05-11 Thread Ishan Chattopadhyaya
Done, Adrien. Thanks!

On Thu, May 11, 2017 at 7:34 PM, Adrien Grand <jpou...@gmail.com> wrote:

> Ishan, wdyt about running addVersion on the branch_6x/master as well? I
> think it would help realize that the 6.6 branch has been cut when looking
> at the CHANGES.txt file and not forget about backporting?
>
> Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <
> ichattopadhy...@gmail.com> a écrit :
>
>> I've cut the branch for 6.6. (branch_6_6). Please feel free to add
>> bugfixes to the branch, if needed.
>> Planning to build the first RC on 15 May. Please let me know if there are
>> any objections.
>>
>> Thanks,
>> Ishan
>>
>> On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
>> ichattopadhy...@gmail.com> wrote:
>>
>>> Planning to cut the release branch in another 10-12 hours.
>>>
>>> On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>>> wrote:
>>>
>>>> i was wondering if there was a specific test for SpanPayloadCheckQuery
>>>> method
>>>>
>>>> matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>>
>>>>
>>>> the only implementation i could locate was in collectLeaf of
>>>> SpanPayloadCheckQuery
>>>>
>>>>
>>>> I will submit JIRA with Patch
>>>>
>>>>
>>>> as a CS *dinosaur* I am more familiar with LISP for dissecting sentence
>>>> fragments (what we called phenomes) than current SEO implementations
>>>>
>>>>
>>>> Can you suggest a book to read to better understand Lucenes pattern
>>>> dissection and match algorithms?
>>>>
>>>>
>>>> Many Thanks for helpful guidance
>>>> Martin
>>>> __
>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> *From:* Erik Hatcher <erik.hatc...@gmail.com>
>>>> *Sent:* Sunday, April 30, 2017 2:06 PM
>>>>
>>>> *To:* dev@lucene.apache.org
>>>> *Subject:* Re: Release 6.6
>>>>
>>>> Martin -
>>>>
>>>> I have to admit to still being unsure what the gist is here - is there
>>>> a bug?   Apologies for not catching what you’re saying/showing here.
>>>> Again, as far as I can tell SpanPayloadCheckQuery is working as expected
>>>> now.
>>>>
>>>> I’m going to focus purely on SOLR-1485 this week to get it committed
>>>> for 6.6.  If there is an issue to address with your work would you please
>>>> open a JIRA and include your patch there?
>>>>
>>>> Thanks,
>>>> Erik
>>>>
>>>>
>>>> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com> wrote:
>>>>
>>>> Mornin' Erik
>>>>
>>>> there is a collectLeaf  override in org.apache.lucene.queries.payloads.
>>>> TestPayloadSpans ..but its never called:
>>>>
>>>> static class VerifyingCollector implements SpanCollector {
>>>> List payloads = new ArrayList<>();
>>>> @Override
>>>> public void collectLeaf(PostingsEnum postings, int position, Term
>>>> term) throws IOException {
>>>>  
>>>>  }
>>>> }
>>>>
>>>>
>>>> the modification in 
>>>> org.apache.lucene.queries.payloads.TestPayloadCheckQuery tests
>>>> collectLeaf for query
>>>>
>>>> //initialise term
>>>>
>>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 231 before
>>>> term1=new org.apache.lucene.index.Term('field','withPayload')");
>>>> org.apache.lucene.index.Term term1=new 
>>>> org.apache.lucene.index.Term("field",
>>>> "withPayload");
>>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 233
>>>> term1="+term1);
>>>>
>>>> //assume position is 5
>>>> int position=5;
>>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 235
>>>> position="+position);
>>>>
>>>> BytesRef pay = new BytesRef("pos: " + position);
>>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 237
>>>> pay="+pay);
>>>>
>>>> //build spanQuery 

Re: Release 6.6

2017-05-11 Thread Adrien Grand
Ishan, wdyt about running addVersion on the branch_6x/master as well? I
think it would help realize that the 6.6 branch has been cut when looking
at the CHANGES.txt file and not forget about backporting?

Le mar. 9 mai 2017 à 17:58, Ishan Chattopadhyaya <ichattopadhy...@gmail.com>
a écrit :

> I've cut the branch for 6.6. (branch_6_6). Please feel free to add
> bugfixes to the branch, if needed.
> Planning to build the first RC on 15 May. Please let me know if there are
> any objections.
>
> Thanks,
> Ishan
>
> On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
> ichattopadhy...@gmail.com> wrote:
>
>> Planning to cut the release branch in another 10-12 hours.
>>
>> On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com>
>> wrote:
>>
>>> i was wondering if there was a specific test for SpanPayloadCheckQuery
>>> method
>>>
>>> matches = payloadToMatch.get(upto).bytesEquals(payload);
>>>
>>>
>>> the only implementation i could locate was in collectLeaf of
>>> SpanPayloadCheckQuery
>>>
>>>
>>> I will submit JIRA with Patch
>>>
>>>
>>> as a CS *dinosaur* I am more familiar with LISP for dissecting sentence
>>> fragments (what we called phenomes) than current SEO implementations
>>>
>>>
>>> Can you suggest a book to read to better understand Lucenes pattern
>>> dissection and match algorithms?
>>>
>>>
>>> Many Thanks for helpful guidance
>>> Martin
>>> __
>>>
>>>
>>>
>>>
>>> --
>>> *From:* Erik Hatcher <erik.hatc...@gmail.com>
>>> *Sent:* Sunday, April 30, 2017 2:06 PM
>>>
>>> *To:* dev@lucene.apache.org
>>> *Subject:* Re: Release 6.6
>>>
>>> Martin -
>>>
>>> I have to admit to still being unsure what the gist is here - is there a
>>> bug?   Apologies for not catching what you’re saying/showing here.  Again,
>>> as far as I can tell SpanPayloadCheckQuery is working as expected now.
>>>
>>> I’m going to focus purely on SOLR-1485 this week to get it committed for
>>> 6.6.  If there is an issue to address with your work would you please open
>>> a JIRA and include your patch there?
>>>
>>> Thanks,
>>> Erik
>>>
>>>
>>> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com> wrote:
>>>
>>> Mornin' Erik
>>>
>>> there is a collectLeaf  override in 
>>> org.apache.lucene.queries.payloads.TestPayloadSpans
>>> ..but its never called:
>>>
>>> static class VerifyingCollector implements SpanCollector {
>>> List payloads = new ArrayList<>();
>>> @Override
>>> public void collectLeaf(PostingsEnum postings, int position, Term
>>> term) throws IOException {
>>>  
>>>  }
>>> }
>>>
>>>
>>> the modification in
>>> org.apache.lucene.queries.payloads.TestPayloadCheckQuery tests collectLeaf
>>> for query
>>>
>>> //initialise term
>>>
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 231 before
>>> term1=new org.apache.lucene.index.Term('field','withPayload')");
>>> org.apache.lucene.index.Term term1=new
>>> org.apache.lucene.index.Term("field", "withPayload");
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 233
>>> term1="+term1);
>>>
>>> //assume position is 5
>>> int position=5;
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 235
>>> position="+position);
>>>
>>> BytesRef pay = new BytesRef("pos: " + position);
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 237
>>> pay="+pay);
>>>
>>> //build spanQuery with term parameter
>>> org.apache.lucene.search.spans.SpanQuery spanQuery1 = new
>>> SpanTermQuery(term1);
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 239
>>> spanQuery1="+spanQuery1);
>>>
>>> //add BytesRef to payloadToMatch list
>>> java.util.List payloadToMatch=new
>>> java.util.ArrayList();
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 241
>>> payloadToMatch="+payloadToMatch);
>>> pay

Re: Release 6.6

2017-05-09 Thread Ishan Chattopadhyaya
I've cut the branch for 6.6. (branch_6_6). Please feel free to add bugfixes
to the branch, if needed.
Planning to build the first RC on 15 May. Please let me know if there are
any objections.

Thanks,
Ishan

On Tue, May 9, 2017 at 11:10 AM, Ishan Chattopadhyaya <
ichattopadhy...@gmail.com> wrote:

> Planning to cut the release branch in another 10-12 hours.
>
> On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com> wrote:
>
>> i was wondering if there was a specific test for SpanPayloadCheckQuery
>> method
>>
>> matches = payloadToMatch.get(upto).bytesEquals(payload);
>>
>>
>> the only implementation i could locate was in collectLeaf of
>> SpanPayloadCheckQuery
>>
>>
>> I will submit JIRA with Patch
>>
>>
>> as a CS *dinosaur* I am more familiar with LISP for dissecting sentence
>> fragments (what we called phenomes) than current SEO implementations
>>
>>
>> Can you suggest a book to read to better understand Lucenes pattern
>> dissection and match algorithms?
>>
>>
>> Many Thanks for helpful guidance
>> Martin
>> __
>>
>>
>>
>>
>> --
>> *From:* Erik Hatcher <erik.hatc...@gmail.com>
>> *Sent:* Sunday, April 30, 2017 2:06 PM
>>
>> *To:* dev@lucene.apache.org
>> *Subject:* Re: Release 6.6
>>
>> Martin -
>>
>> I have to admit to still being unsure what the gist is here - is there a
>> bug?   Apologies for not catching what you’re saying/showing here.  Again,
>> as far as I can tell SpanPayloadCheckQuery is working as expected now.
>>
>> I’m going to focus purely on SOLR-1485 this week to get it committed for
>> 6.6.  If there is an issue to address with your work would you please open
>> a JIRA and include your patch there?
>>
>> Thanks,
>> Erik
>>
>>
>> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com> wrote:
>>
>> Mornin' Erik
>>
>> there is a collectLeaf  override in org.apache.lucene.queries.payloads.
>> TestPayloadSpans ..but its never called:
>>
>> static class VerifyingCollector implements SpanCollector {
>> List payloads = new ArrayList<>();
>> @Override
>> public void collectLeaf(PostingsEnum postings, int position, Term
>> term) throws IOException {
>>  
>>  }
>> }
>>
>>
>> the modification in org.apache.lucene.queries.payl
>> oads.TestPayloadCheckQuery tests collectLeaf for query
>>
>> //initialise term
>>
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 231 before
>> term1=new org.apache.lucene.index.Term('field','withPayload')");
>> org.apache.lucene.index.Term term1=new org.apache.lucene.index.Term("field",
>> "withPayload");
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 233
>> term1="+term1);
>>
>> //assume position is 5
>> int position=5;
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 235
>> position="+position);
>>
>> BytesRef pay = new BytesRef("pos: " + position);
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 237
>> pay="+pay);
>>
>> //build spanQuery with term parameter
>> org.apache.lucene.search.spans.SpanQuery spanQuery1 = new
>> SpanTermQuery(term1);
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 239
>> spanQuery1="+spanQuery1);
>>
>> //add BytesRef to payloadToMatch list
>> java.util.List payloadToMatch=new
>> java.util.ArrayList();
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 241
>> payloadToMatch="+payloadToMatch);
>> payloadToMatch.add(pay);
>>
>> //build SpanPayloadCheckQuery
>>
>> query=new org.apache.lucene.queries.payloads.SpanPayloadCheckQuery(
>> (org.apache.lucene.search.spans.SpanQuery)query,
>> (java.util.List)payloadToMatch);
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 249
>> query="+query);
>>
>> //build lucene Directory (TODO: we should use an existing directory with
>> REAL test-data)
>> org.apache.lucene.store.Directory ram = newDirectory(com.carrotsearch.
>> randomizedtesting.RandomizedContext.current().getRandom());
>>
>> //build SegmentReader from Directory
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 251
>> ram="+ram);
>> Se

Re: Release 6.6

2017-05-08 Thread Ishan Chattopadhyaya
Planning to cut the release branch in another 10-12 hours.

On Mon, May 1, 2017 at 4:00 PM, Martin Gainty <mgai...@hotmail.com> wrote:

> i was wondering if there was a specific test for SpanPayloadCheckQuery
> method
>
> matches = payloadToMatch.get(upto).bytesEquals(payload);
>
>
> the only implementation i could locate was in collectLeaf of
> SpanPayloadCheckQuery
>
>
> I will submit JIRA with Patch
>
>
> as a CS *dinosaur* I am more familiar with LISP for dissecting sentence
> fragments (what we called phenomes) than current SEO implementations
>
>
> Can you suggest a book to read to better understand Lucenes pattern
> dissection and match algorithms?
>
>
> Many Thanks for helpful guidance
> Martin
> __
>
>
>
>
> --
> *From:* Erik Hatcher <erik.hatc...@gmail.com>
> *Sent:* Sunday, April 30, 2017 2:06 PM
>
> *To:* dev@lucene.apache.org
> *Subject:* Re: Release 6.6
>
> Martin -
>
> I have to admit to still being unsure what the gist is here - is there a
> bug?   Apologies for not catching what you’re saying/showing here.  Again,
> as far as I can tell SpanPayloadCheckQuery is working as expected now.
>
> I’m going to focus purely on SOLR-1485 this week to get it committed for
> 6.6.  If there is an issue to address with your work would you please open
> a JIRA and include your patch there?
>
> Thanks,
> Erik
>
>
> On Apr 30, 2017, at 7:47 AM, Martin Gainty <mgai...@hotmail.com> wrote:
>
> Mornin' Erik
>
> there is a collectLeaf  override in org.apache.lucene.queries.payloads.
> TestPayloadSpans ..but its never called:
>
> static class VerifyingCollector implements SpanCollector {
> List payloads = new ArrayList<>();
> @Override
> public void collectLeaf(PostingsEnum postings, int position, Term
> term) throws IOException {
>  
>  }
> }
>
>
> the modification in org.apache.lucene.queries.payloads.TestPayloadCheckQuery 
> tests
> collectLeaf for query
>
> //initialise term
>
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 231 before
> term1=new org.apache.lucene.index.Term('field','withPayload')");
> org.apache.lucene.index.Term term1=new org.apache.lucene.index.Term("field",
> "withPayload");
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 233
> term1="+term1);
>
> //assume position is 5
> int position=5;
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 235
> position="+position);
>
> BytesRef pay = new BytesRef("pos: " + position);
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 237
> pay="+pay);
>
> //build spanQuery with term parameter
> org.apache.lucene.search.spans.SpanQuery spanQuery1 = new
> SpanTermQuery(term1);
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 239
> spanQuery1="+spanQuery1);
>
> //add BytesRef to payloadToMatch list
> java.util.List payloadToMatch=new
> java.util.ArrayList();
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 241
> payloadToMatch="+payloadToMatch);
> payloadToMatch.add(pay);
>
> //build SpanPayloadCheckQuery
>
> query=new org.apache.lucene.queries.payloads.SpanPayloadCheckQuery(
> (org.apache.lucene.search.spans.SpanQuery)query,
> (java.util.List)payloadToMatch);
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 249
> query="+query);
>
> //build lucene Directory (TODO: we should use an existing directory with
> REAL test-data)
> org.apache.lucene.store.Directory ram = newDirectory(com.carrotsearch.
> randomizedtesting.RandomizedContext.current().getRandom());
>
> //build SegmentReader from Directory
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 251
> ram="+ram);
> SegmentReader reader = getOnlySegmentReader(org.apache.lucene.index.
> DirectoryReader.open(ram));
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 253
> reader="+reader);
>
> //populate SlowCompositeReaderWrapper with SegmentReader
> org.apache.lucene.index.LeafReader sr = org.apache.lucene.index.
> SlowCompositeReaderWrapper.wrap(reader);
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 255
> sr="+sr);
>
> //add term to SegmentReader postings
> org.apache.lucene.index.PostingsEnum postings = sr.postings(term1,
> PostingsEnum.PAYLOADS);
>
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 257 before
> query.getPayloadChecker().collectLeaf((org.apache

Re: Release 6.6

2017-05-01 Thread Martin Gainty
i was wondering if there was a specific test for SpanPayloadCheckQuery method


matches = payloadToMatch.get(upto).bytesEquals(payload);


the only implementation i could locate was in collectLeaf of 
SpanPayloadCheckQuery


I will submit JIRA with Patch


as a CS *dinosaur* I am more familiar with LISP for dissecting sentence 
fragments (what we called phenomes) than current SEO implementations


Can you suggest a book to read to better understand Lucenes pattern dissection 
and match algorithms?


Many Thanks for helpful guidance

Martin
__




From: Erik Hatcher <erik.hatc...@gmail.com>
Sent: Sunday, April 30, 2017 2:06 PM
To: dev@lucene.apache.org
Subject: Re: Release 6.6

Martin -

I have to admit to still being unsure what the gist is here - is there a bug?   
Apologies for not catching what you’re saying/showing here.  Again, as far as I 
can tell SpanPayloadCheckQuery is working as expected now.

I’m going to focus purely on SOLR-1485 this week to get it committed for 6.6.  
If there is an issue to address with your work would you please open a JIRA and 
include your patch there?

Thanks,
Erik


On Apr 30, 2017, at 7:47 AM, Martin Gainty 
<mgai...@hotmail.com<mailto:mgai...@hotmail.com>> wrote:

Mornin' Erik

there is a collectLeaf  override in 
org.apache.lucene.queries.payloads.TestPayloadSpans ..but its never called:

static class VerifyingCollector implements SpanCollector {
List payloads = new ArrayList<>();
@Override
public void collectLeaf(PostingsEnum postings, int position, Term term) 
throws IOException {
 
 }
}

the modification in org.apache.lucene.queries.payloads.TestPayloadCheckQuery 
tests collectLeaf for query

//initialise term

log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 231 before 
term1=new org.apache.lucene.index.Term('field','withPayload')");
org.apache.lucene.index.Term term1=new org.apache.lucene.index.Term("field", 
"withPayload");
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 233 term1="+term1);

//assume position is 5
int position=5;
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 235 
position="+position);

BytesRef pay = new BytesRef("pos: " + position);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 237 pay="+pay);

//build spanQuery with term parameter
org.apache.lucene.search.spans.SpanQuery spanQuery1 = new 
SpanTermQuery(term1);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 239 
spanQuery1="+spanQuery1);

//add BytesRef to payloadToMatch list
java.util.List payloadToMatch=new 
java.util.ArrayList();
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 241 
payloadToMatch="+payloadToMatch);
payloadToMatch.add(pay);


//build SpanPayloadCheckQuery

query=new org.apache.lucene.queries.payloads.SpanPayloadCheckQuery(
(org.apache.lucene.search.spans.SpanQuery)query,
(java.util.List)payloadToMatch);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 249 query="+query);

//build lucene Directory (TODO: we should use an existing directory with REAL 
test-data)
org.apache.lucene.store.Directory ram = 
newDirectory(com.carrotsearch.randomizedtesting.RandomizedContext.current().getRandom());

//build SegmentReader from Directory
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 251 ram="+ram);
SegmentReader reader = 
getOnlySegmentReader(org.apache.lucene.index.DirectoryReader.open(ram));
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 253 
reader="+reader);

//populate SlowCompositeReaderWrapper with SegmentReader
org.apache.lucene.index.LeafReader sr = 
org.apache.lucene.index.SlowCompositeReaderWrapper.wrap(reader);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 255 sr="+sr);

//add term to SegmentReader postings
org.apache.lucene.index.PostingsEnum postings = sr.postings(term1, 
PostingsEnum.PAYLOADS);

log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 257 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where postings="+postings);

//display the parameters for collectLeaf method for query:
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 258 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where position="+position);

log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 259 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where term1="+term1);
try
{ //public void collectLeaf(org.apache.lucene.index.Po

Re: Release 6.6

2017-04-30 Thread Erik Hatcher
bug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 264 
> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>  (int)position,(org.apache.lucene.index.Term)term1) LINE 106 throws 
> IOException ="+ioe.getMessage()); }
> 
> collectLeaf is the only method that compares 
> matches=payloadToMatch.get(upto).bytesEquals(payload)
> to determine "match"
> 
> unless of course match between individual payload and payloadToMatch is 
> tested elsewhere ?
> 
> WDYT?
> Martin 
> __ 
> 
> 
> 
> From: Erik Hatcher <erik.hatc...@gmail.com>
> Sent: Saturday, April 29, 2017 7:58 PM
> To: Lucene/Solr dev
> Subject: Re: Release 6.6
>  
> Martin -
> 
> Interesting test but I couldn’t copy/paste it in to try it out as it uses 
> some logging and APIs that aren’t already in the project and classpath of 
> these lucene tests within that queries module (within IntelliJ, mind you).
> 
> I was able to resolve the misunderstanding (and .equals/.hashCode bugs) so 
> everything is working as expected with SpanPayloadCheckQuery for me now.   Do 
> your tests point out an issue?   Or confirming that it is working properly at 
> a lower level?
> 
> Erik
> 
> 
>> On Apr 29, 2017, at 9:08 AM, Martin Gainty <mgai...@hotmail.com 
>> <mailto:mgai...@hotmail.com>> wrote:
>> 
>> Martin Gainty has shared a OneDrive file with you. To view it, click the 
>> link below.
>>  
>> TestPayloadCheckQuery.java
>>  <https://1drv.ms/u/s!AkpuiYcNg4cSgWRHc6DadiCIYaFN> 
>> <https://1drv.ms/u/s!AkpuiYcNg4cSgWRHc6DadiCIYaFN>   
>> TestPayloadCheckQuery.java <https://1drv.ms/u/s!AkpuiYcNg4cSgWRHc6DadiCIYaFN>
>> attached
>> 
>> I coded this last nite so it is "quick and dirty"
>> 
>> in any case let me know if  testSpanPayloadCheck() method modification 
>> properly addresses your situation
>> 
>> Thanks!
>> Martin 
>> __ 
>> 
>> 
>> 
>> From: Erik Hatcher <erik.hatc...@gmail.com <mailto:erik.hatc...@gmail.com>>
>> Sent: Saturday, April 29, 2017 8:58 AM
>> To: dev@lucene.apache.org <mailto:dev@lucene.apache.org>
>> Subject: Re: Release 6.6
>>  
>> Martin - there is a test class specifically for this query.   See the recent 
>> commits I've made to test it further fixing .equals/.hashCode and rewrite. 
>> 
>> Can you send your full test code?
>> 
>>Erik
>> 
>> On Apr 29, 2017, at 07:32, Martin Gainty <mgai...@hotmail.com 
>> <mailto:mgai...@hotmail.com>> wrote:
>> 
>>> when Erik mentioned he couldnt get SpanPayloadCheckQuery to work I was 
>>> about to reply just run that TestCase
>>> (until i discovered there wasnt any!)
>>> 
>>> i'll start the bidding at 1 dinar for TestCase 
>>> org.apache.lucene.queries.payloads.TestPayloadCheckQuery mod:
>>>   @org.junit.Test
>>>   public void testSpanPayloadCheck() throws Exception
>>> 
>>> //now lets test the collectLeaf for query
>>> //of course calling Base Class SpanPayloadCheckQuery to construct query
>>> 
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 257 before 
>>> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>>>  (int)position,(org.apache.lucene.index.Term)term1) where 
>>> postings="+postings);
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 258 before 
>>> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>>>  (int)position,(org.apache.lucene.index.Term)term1) where 
>>> position="+position);
>>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 259 before 
>>> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>>>  (int)position,(org.apache.lucene.index.Term)term1) where term1="+term1);
>>> 
>>> try
>>> { //test public void collectLeaf(org.apache.lucene.index.PostingsEnum 
>>> postings, int position,  //org.apache.lucene.index.Term term) 
>>> throws java.io.IOException {
>>> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>>>  (int)position,(org.apache.lucene.index.Term)term1);
>>> }
>>> catch(java.io.IOException ioe) { 
>>> log.error("TestPayloadCheckQuery::testSpanPayloadCheck LINE 264 
>>> query.getPayloadChecke

Re: Release 6.6

2017-04-30 Thread Martin Gainty
Mornin' Erik

there is a collectLeaf  override in 
org.apache.lucene.queries.payloads.TestPayloadSpans ..but its never called:


static class VerifyingCollector implements SpanCollector {
List payloads = new ArrayList<>();
@Override
public void collectLeaf(PostingsEnum postings, int position, Term term) 
throws IOException {
 
 }
}


the modification in org.apache.lucene.queries.payloads.TestPayloadCheckQuery 
tests collectLeaf for query


//initialise term

log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 231 before 
term1=new org.apache.lucene.index.Term('field','withPayload')");
org.apache.lucene.index.Term term1=new org.apache.lucene.index.Term("field", 
"withPayload");
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 233 term1="+term1);

//assume position is 5
int position=5;
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 235 
position="+position);

BytesRef pay = new BytesRef("pos: " + position);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 237 pay="+pay);

//build spanQuery with term parameter
org.apache.lucene.search.spans.SpanQuery spanQuery1 = new 
SpanTermQuery(term1);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 239 
spanQuery1="+spanQuery1);

//add BytesRef to payloadToMatch list
java.util.List payloadToMatch=new 
java.util.ArrayList();
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 241 
payloadToMatch="+payloadToMatch);
payloadToMatch.add(pay);


//build SpanPayloadCheckQuery

query=new org.apache.lucene.queries.payloads.SpanPayloadCheckQuery(
(org.apache.lucene.search.spans.SpanQuery)query,
(java.util.List)payloadToMatch);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 249 query="+query);

//build lucene Directory (TODO: we should use an existing directory with REAL 
test-data)
org.apache.lucene.store.Directory ram = 
newDirectory(com.carrotsearch.randomizedtesting.RandomizedContext.current().getRandom());

//build SegmentReader from Directory
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 251 ram="+ram);
SegmentReader reader = 
getOnlySegmentReader(org.apache.lucene.index.DirectoryReader.open(ram));
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 253 
reader="+reader);

//populate SlowCompositeReaderWrapper with SegmentReader
org.apache.lucene.index.LeafReader sr = 
org.apache.lucene.index.SlowCompositeReaderWrapper.wrap(reader);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 255 sr="+sr);

//add term to SegmentReader postings
org.apache.lucene.index.PostingsEnum postings = sr.postings(term1, 
PostingsEnum.PAYLOADS);

log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 257 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where postings="+postings);

//display the parameters for collectLeaf method for query:
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 258 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where position="+position);

log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 259 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where term1="+term1);
try
{ //public void collectLeaf(org.apache.lucene.index.PostingsEnum postings, 
int position,   //org.apache.lucene.index.Term term) throws 
java.io.IOException {
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1);
}
catch(java.io.IOException ioe) { 
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 264 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) LINE 106 throws IOException 
="+ioe.getMessage()); }

collectLeaf is the only method that compares 
matches=payloadToMatch.get(upto).bytesEquals(payload)

to determine "match"


unless of course match between individual payload and payloadToMatch is tested 
elsewhere ?


WDYT?

Martin
__________




From: Erik Hatcher <erik.hatc...@gmail.com>
Sent: Saturday, April 29, 2017 7:58 PM
To: Lucene/Solr dev
Subject: Re: Release 6.6

Martin -

Interesting test but I couldn’t copy/paste it in to try it out as it uses some 
logging and APIs that aren’t already in the project and classpath of these 
lucene tests within that queries module (within IntelliJ, mind you).

I was able to resolve the misunderstanding (and .equals/.hashCode bugs) so

Re: Release 6.6

2017-04-29 Thread Erik Hatcher
Martin -

Interesting test but I couldn’t copy/paste it in to try it out as it uses some 
logging and APIs that aren’t already in the project and classpath of these 
lucene tests within that queries module (within IntelliJ, mind you).

I was able to resolve the misunderstanding (and .equals/.hashCode bugs) so 
everything is working as expected with SpanPayloadCheckQuery for me now.   Do 
your tests point out an issue?   Or confirming that it is working properly at a 
lower level?

Erik


> On Apr 29, 2017, at 9:08 AM, Martin Gainty <mgai...@hotmail.com> wrote:
> 
> Martin Gainty has shared a OneDrive file with you. To view it, click the link 
> below.
>   
> TestPayloadCheckQuery.java
>  <https://1drv.ms/u/s!AkpuiYcNg4cSgWRHc6DadiCIYaFN> 
> <https://1drv.ms/u/s!AkpuiYcNg4cSgWRHc6DadiCIYaFN>
> TestPayloadCheckQuery.java <https://1drv.ms/u/s!AkpuiYcNg4cSgWRHc6DadiCIYaFN>
> attached
> 
> I coded this last nite so it is "quick and dirty"
> 
> in any case let me know if  testSpanPayloadCheck() method modification 
> properly addresses your situation
> 
> Thanks!
> Martin 
> __ 
> 
> 
> 
> From: Erik Hatcher <erik.hatc...@gmail.com>
> Sent: Saturday, April 29, 2017 8:58 AM
> To: dev@lucene.apache.org
> Subject: Re: Release 6.6
>  
> Martin - there is a test class specifically for this query.   See the recent 
> commits I've made to test it further fixing .equals/.hashCode and rewrite. 
> 
> Can you send your full test code?
> 
>Erik
> 
> On Apr 29, 2017, at 07:32, Martin Gainty <mgai...@hotmail.com 
> <mailto:mgai...@hotmail.com>> wrote:
> 
>> when Erik mentioned he couldnt get SpanPayloadCheckQuery to work I was about 
>> to reply just run that TestCase
>> (until i discovered there wasnt any!)
>> 
>> i'll start the bidding at 1 dinar for TestCase 
>> org.apache.lucene.queries.payloads.TestPayloadCheckQuery mod:
>>   @org.junit.Test
>>   public void testSpanPayloadCheck() throws Exception
>> 
>> //now lets test the collectLeaf for query
>> //of course calling Base Class SpanPayloadCheckQuery to construct query
>> 
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 257 before 
>> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>>  (int)position,(org.apache.lucene.index.Term)term1) where 
>> postings="+postings);
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 258 before 
>> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>>  (int)position,(org.apache.lucene.index.Term)term1) where 
>> position="+position);
>> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 259 before 
>> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>>  (int)position,(org.apache.lucene.index.Term)term1) where term1="+term1);
>> 
>> try
>> { //test public void collectLeaf(org.apache.lucene.index.PostingsEnum 
>> postings, int position,  //org.apache.lucene.index.Term term) 
>> throws java.io.IOException {
>> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>>  (int)position,(org.apache.lucene.index.Term)term1);
>> }
>> catch(java.io.IOException ioe) { 
>> log.error("TestPayloadCheckQuery::testSpanPayloadCheck LINE 264 
>> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>>  (int)position,(org.apache.lucene.index.Term)term1) LINE 106 throws 
>> IOException ="+ioe.getMessage()); }
>> 
>> unless of course anyone has a better idea to test collectLeaf ?
>> Martin 
>> __ 
>> 
>> 
>> 
>> From: Erik Hatcher <erik.hatc...@gmail.com <mailto:erik.hatc...@gmail.com>>
>> Sent: Tuesday, April 25, 2017 7:57 PM
>> To: dev@lucene.apache.org <mailto:dev@lucene.apache.org>
>> Subject: Re: Release 6.6
>>  
>> Probably no bribe needed, but an updated patch would be a good start (or 
>> will the 2.5 year old patch still apply and be acceptable as-is?)
>> 
>> Erik
>> 
>>> On Apr 25, 2017, at 7:52 PM, Walter Underwood <wun...@wunderwood.org 
>>> <mailto:wun...@wunderwood.org>> wrote:
>>> 
>>> Who do I have to bribe to get SOLR-629 included?
>>> 
>>> https://issues.apache.org/jira/browse/SOLR-629 
>>> <https://issues.apache.org/jira/browse/SOLR-629>
>>> 
>>> wunder
>>

Re: Release 6.6

2017-04-29 Thread Martin Gainty
Martin Gainty has shared a OneDrive file with you. To view it, click the link 
below.


<https://1drv.ms/u/s!AkpuiYcNg4cSgWRHc6DadiCIYaFN>
[https://r1.res.office365.com/owa/prem/images/dc-generic_20.png]<https://1drv.ms/u/s!AkpuiYcNg4cSgWRHc6DadiCIYaFN>

TestPayloadCheckQuery.java<https://1drv.ms/u/s!AkpuiYcNg4cSgWRHc6DadiCIYaFN>




attached


I coded this last nite so it is "quick and dirty"

in any case let me know if  testSpanPayloadCheck() method modification properly 
addresses your situation


Thanks!

Martin
__




From: Erik Hatcher <erik.hatc...@gmail.com>
Sent: Saturday, April 29, 2017 8:58 AM
To: dev@lucene.apache.org
Subject: Re: Release 6.6

Martin - there is a test class specifically for this query.   See the recent 
commits I've made to test it further fixing .equals/.hashCode and rewrite.

Can you send your full test code?

   Erik

On Apr 29, 2017, at 07:32, Martin Gainty 
<mgai...@hotmail.com<mailto:mgai...@hotmail.com>> wrote:


when Erik mentioned he couldnt get SpanPayloadCheckQuery to work I was about to 
reply just run that TestCase
(until i discovered there wasnt any!)


i'll start the bidding at 1 dinar for TestCase 
org.apache.lucene.queries.payloads.TestPayloadCheckQuery mod:

  @org.junit.Test
  public void testSpanPayloadCheck() throws Exception


//now lets test the collectLeaf for query
//of course calling Base Class SpanPayloadCheckQuery to construct query

log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 257 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where postings="+postings);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 258 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where position="+position);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 259 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where term1="+term1);

try
{ //test public void collectLeaf(org.apache.lucene.index.PostingsEnum 
postings, int position,  //org.apache.lucene.index.Term term) 
throws java.io.IOException {
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1);
}
catch(java.io.IOException ioe) { 
log.error("TestPayloadCheckQuery::testSpanPayloadCheck LINE 264 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) LINE 106 throws IOException 
="+ioe.getMessage()); }


unless of course anyone has a better idea to test collectLeaf ?

Martin
__




From: Erik Hatcher <erik.hatc...@gmail.com<mailto:erik.hatc...@gmail.com>>
Sent: Tuesday, April 25, 2017 7:57 PM
To: dev@lucene.apache.org<mailto:dev@lucene.apache.org>
Subject: Re: Release 6.6

Probably no bribe needed, but an updated patch would be a good start (or will 
the 2.5 year old patch still apply and be acceptable as-is?)

Erik

On Apr 25, 2017, at 7:52 PM, Walter Underwood 
<wun...@wunderwood.org<mailto:wun...@wunderwood.org>> wrote:

Who do I have to bribe to get SOLR-629 included?

https://issues.apache.org/jira/browse/SOLR-629

wunder
Walter Underwood
wun...@wunderwood.org<mailto:wun...@wunderwood.org>
http://observer.wunderwood.org/  (my blog)


On Apr 25, 2017, at 10:46 AM, Ishan Chattopadhyaya 
<ichattopadhy...@gmail.com<mailto:ichattopadhy...@gmail.com>> wrote:

Hi,
We have lots of changes, optimizations, bug fixes for 6.6. I'd like to propose 
a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0 release).

I can volunteer to release this, and I can cut the release branch around 4th 
May, in order to let some time for devs to finish current issues.

What do you think?

Regards,
Ishan





Re: Release 6.6

2017-04-29 Thread Erik Hatcher
Martin - there is a test class specifically for this query.   See the recent 
commits I've made to test it further fixing .equals/.hashCode and rewrite. 

Can you send your full test code?

   Erik

> On Apr 29, 2017, at 07:32, Martin Gainty <mgai...@hotmail.com> wrote:
> 
> when Erik mentioned he couldnt get SpanPayloadCheckQuery to work I was about 
> to reply just run that TestCase
> (until i discovered there wasnt any!)
> 
> 
> i'll start the bidding at 1 dinar for TestCase 
> org.apache.lucene.queries.payloads.TestPayloadCheckQuery mod:
> 
>   @org.junit.Test
>   public void testSpanPayloadCheck() throws Exception
> 
> 
> 
> //now lets test the collectLeaf for query
> //of course calling Base Class SpanPayloadCheckQuery to construct query
> 
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 257 before 
> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>  (int)position,(org.apache.lucene.index.Term)term1) where 
> postings="+postings);
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 258 before 
> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>  (int)position,(org.apache.lucene.index.Term)term1) where 
> position="+position);
> log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 259 before 
> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>  (int)position,(org.apache.lucene.index.Term)term1) where term1="+term1);
> 
> try
> { //test public void collectLeaf(org.apache.lucene.index.PostingsEnum 
> postings, int position,  //org.apache.lucene.index.Term term) 
> throws java.io.IOException {
> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>  (int)position,(org.apache.lucene.index.Term)term1);
> }
> catch(java.io.IOException ioe) { 
> log.error("TestPayloadCheckQuery::testSpanPayloadCheck LINE 264 
> query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
>  (int)position,(org.apache.lucene.index.Term)term1) LINE 106 throws 
> IOException ="+ioe.getMessage()); }
> 
> unless of course anyone has a better idea to test collectLeaf ?
> 
> Martin 
> ______ 
> 
> 
> 
> From: Erik Hatcher <erik.hatc...@gmail.com>
> Sent: Tuesday, April 25, 2017 7:57 PM
> To: dev@lucene.apache.org
> Subject: Re: Release 6.6
>  
> Probably no bribe needed, but an updated patch would be a good start (or will 
> the 2.5 year old patch still apply and be acceptable as-is?)
> 
> Erik
> 
>> On Apr 25, 2017, at 7:52 PM, Walter Underwood <wun...@wunderwood.org> wrote:
>> 
>> Who do I have to bribe to get SOLR-629 included?
>> 
>> https://issues.apache.org/jira/browse/SOLR-629
>> 
>> wunder
>> Walter Underwood
>> wun...@wunderwood.org
>> http://observer.wunderwood.org/  (my blog)
>> 
>> 
>>> On Apr 25, 2017, at 10:46 AM, Ishan Chattopadhyaya 
>>> <ichattopadhy...@gmail.com> wrote:
>>> 
>>> Hi,
>>> We have lots of changes, optimizations, bug fixes for 6.6. I'd like to 
>>> propose a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0 
>>> release).
>>> 
>>> I can volunteer to release this, and I can cut the release branch around 
>>> 4th May, in order to let some time for devs to finish current issues.
>>> 
>>> What do you think?
>>> 
>>> Regards,
>>> Ishan
>>> 
>> 
> 


Re: Release 6.6

2017-04-29 Thread Martin Gainty
when Erik mentioned he couldnt get SpanPayloadCheckQuery to work I was about to 
reply just run that TestCase
(until i discovered there wasnt any!)


i'll start the bidding at 1 dinar for TestCase 
org.apache.lucene.queries.payloads.TestPayloadCheckQuery mod:

  @org.junit.Test
  public void testSpanPayloadCheck() throws Exception


//now lets test the collectLeaf for query
//of course calling Base Class SpanPayloadCheckQuery to construct query

log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 257 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where postings="+postings);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 258 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where position="+position);
log.debug("TestPayloadCheckQuery::testSpanPayloadCheck LINE 259 before 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) where term1="+term1);

try
{ //test public void collectLeaf(org.apache.lucene.index.PostingsEnum 
postings, int position,  //org.apache.lucene.index.Term term) 
throws java.io.IOException {
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1);
}
catch(java.io.IOException ioe) { 
log.error("TestPayloadCheckQuery::testSpanPayloadCheck LINE 264 
query.getPayloadChecker().collectLeaf((org.apache.lucene.index.PostingsEnum)postings,
 (int)position,(org.apache.lucene.index.Term)term1) LINE 106 throws IOException 
="+ioe.getMessage()); }


unless of course anyone has a better idea to test collectLeaf ?

Martin
__




From: Erik Hatcher <erik.hatc...@gmail.com>
Sent: Tuesday, April 25, 2017 7:57 PM
To: dev@lucene.apache.org
Subject: Re: Release 6.6

Probably no bribe needed, but an updated patch would be a good start (or will 
the 2.5 year old patch still apply and be acceptable as-is?)

Erik

On Apr 25, 2017, at 7:52 PM, Walter Underwood 
<wun...@wunderwood.org<mailto:wun...@wunderwood.org>> wrote:

Who do I have to bribe to get SOLR-629 included?

https://issues.apache.org/jira/browse/SOLR-629

wunder
Walter Underwood
wun...@wunderwood.org<mailto:wun...@wunderwood.org>
http://observer.wunderwood.org/  (my blog)


On Apr 25, 2017, at 10:46 AM, Ishan Chattopadhyaya 
<ichattopadhy...@gmail.com<mailto:ichattopadhy...@gmail.com>> wrote:

Hi,
We have lots of changes, optimizations, bug fixes for 6.6. I'd like to propose 
a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0 release).

I can volunteer to release this, and I can cut the release branch around 4th 
May, in order to let some time for devs to finish current issues.

What do you think?

Regards,
Ishan





Re: Release 6.6

2017-04-26 Thread David Smiley
+1 to start complaining in logs ASAP.
(Thanks for working on this Jan).

On Wed, Apr 26, 2017 at 5:32 AM Jan Høydahl  wrote:

> I was hoping to nuke the  and solrQueryParser from
> schema https://issues.apache.org/jira/browse/SOLR-7041
> so 7.0.0 could get rid of that code/config completely. But a ton of tests
> still depend on these deprecated configs, and fixing it all in one patch
> was a bit heavy.
> Should we commit the “complaining in logs” part of the patch and then
> convert each test in a series of commits later?
>
> --
> Jan Høydahl, search solution architect
> Cominvent AS - www.cominvent.com
>
> 25. apr. 2017 kl. 19.46 skrev Ishan Chattopadhyaya <
> ichattopadhy...@gmail.com>:
>
> Hi,
> We have lots of changes, optimizations, bug fixes for 6.6. I'd like to
> propose a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0
> release).
>
> I can volunteer to release this, and I can cut the release branch around
> 4th May, in order to let some time for devs to finish current issues.
>
> What do you think?
>
> Regards,
> Ishan
>
>
> --
Lucene/Solr Search Committer, Consultant, Developer, Author, Speaker
LinkedIn: http://linkedin.com/in/davidwsmiley | Book:
http://www.solrenterprisesearchserver.com


Re: Release 6.6

2017-04-26 Thread Jan Høydahl
I was hoping to nuke the  and solrQueryParser from schema 
https://issues.apache.org/jira/browse/SOLR-7041 

so 7.0.0 could get rid of that code/config completely. But a ton of tests still 
depend on these deprecated configs, and fixing it all in one patch was a bit 
heavy.
Should we commit the “complaining in logs” part of the patch and then convert 
each test in a series of commits later?

--
Jan Høydahl, search solution architect
Cominvent AS - www.cominvent.com

> 25. apr. 2017 kl. 19.46 skrev Ishan Chattopadhyaya 
> :
> 
> Hi,
> We have lots of changes, optimizations, bug fixes for 6.6. I'd like to 
> propose a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0 
> release).
> 
> I can volunteer to release this, and I can cut the release branch around 4th 
> May, in order to let some time for devs to finish current issues.
> 
> What do you think?
> 
> Regards,
> Ishan
> 



Re: Release 6.6

2017-04-25 Thread Walter Underwood
I’m a little tired of re-implementing the patch. I did it for 1.3, 3.x, and 
4.x. Perhaps someone more familiar with edismax could take a look.

With the 100X speedup for fuzzy in 4.x, should be widely useful. 

wunder
Walter Underwood
wun...@wunderwood.org
http://observer.wunderwood.org/  (my blog)


> On Apr 25, 2017, at 4:57 PM, Erik Hatcher  wrote:
> 
> Probably no bribe needed, but an updated patch would be a good start (or will 
> the 2.5 year old patch still apply and be acceptable as-is?)
> 
>   Erik
> 
>> On Apr 25, 2017, at 7:52 PM, Walter Underwood > > wrote:
>> 
>> Who do I have to bribe to get SOLR-629 included?
>> 
>> https://issues.apache.org/jira/browse/SOLR-629 
>> 
>> 
>> wunder
>> Walter Underwood
>> wun...@wunderwood.org 
>> http://observer.wunderwood.org/   (my blog)
>> 
>> 
>>> On Apr 25, 2017, at 10:46 AM, Ishan Chattopadhyaya 
>>> > wrote:
>>> 
>>> Hi,
>>> We have lots of changes, optimizations, bug fixes for 6.6. I'd like to 
>>> propose a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0 
>>> release).
>>> 
>>> I can volunteer to release this, and I can cut the release branch around 
>>> 4th May, in order to let some time for devs to finish current issues.
>>> 
>>> What do you think?
>>> 
>>> Regards,
>>> Ishan
>>> 
>> 
> 



Re: Release 6.6

2017-04-25 Thread Erik Hatcher
Probably no bribe needed, but an updated patch would be a good start (or will 
the 2.5 year old patch still apply and be acceptable as-is?)

Erik

> On Apr 25, 2017, at 7:52 PM, Walter Underwood  wrote:
> 
> Who do I have to bribe to get SOLR-629 included?
> 
> https://issues.apache.org/jira/browse/SOLR-629 
> 
> 
> wunder
> Walter Underwood
> wun...@wunderwood.org 
> http://observer.wunderwood.org/  (my blog)
> 
> 
>> On Apr 25, 2017, at 10:46 AM, Ishan Chattopadhyaya 
>> > wrote:
>> 
>> Hi,
>> We have lots of changes, optimizations, bug fixes for 6.6. I'd like to 
>> propose a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0 
>> release).
>> 
>> I can volunteer to release this, and I can cut the release branch around 4th 
>> May, in order to let some time for devs to finish current issues.
>> 
>> What do you think?
>> 
>> Regards,
>> Ishan
>> 
> 



Re: Release 6.6

2017-04-25 Thread Walter Underwood
Who do I have to bribe to get SOLR-629 included?

https://issues.apache.org/jira/browse/SOLR-629 


wunder
Walter Underwood
wun...@wunderwood.org
http://observer.wunderwood.org/  (my blog)


> On Apr 25, 2017, at 10:46 AM, Ishan Chattopadhyaya 
>  wrote:
> 
> Hi,
> We have lots of changes, optimizations, bug fixes for 6.6. I'd like to 
> propose a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0 
> release).
> 
> I can volunteer to release this, and I can cut the release branch around 4th 
> May, in order to let some time for devs to finish current issues.
> 
> What do you think?
> 
> Regards,
> Ishan
> 



Re: Release 6.6

2017-04-25 Thread Christine Poerschke (BLOOMBERG/ LONDON)
I'd appreciate feedback on https://issues.apache.org/jira/browse/SOLR-10479 
with a view towards making it into the 6.6 release.

Christine

- Original Message -
From: dev@lucene.apache.org
To: dev@lucene.apache.org
At: 04/25/17 19:13:39

I’d like to get SOLR-1485, at least the numeric payload value source part, in 
to 6.6.   With some help (nudge nudge, Erickson) I think we can get this in by 
then.

Erik


> On Apr 25, 2017, at 1:46 PM, Ishan Chattopadhyaya  
> wrote:
> 
> Hi,
> We have lots of changes, optimizations, bug fixes for 6.6. I'd like to 
> propose a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0 
> release).
> 
> I can volunteer to release this, and I can cut the release branch around 4th 
> May, in order to let some time for devs to finish current issues.
> 
> What do you think?
> 
> Regards,
> Ishan
> 


-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



Re: Release 6.6

2017-04-25 Thread Erik Hatcher
I’d like to get SOLR-1485, at least the numeric payload value source part, in 
to 6.6.   With some help (nudge nudge, Erickson) I think we can get this in by 
then.

Erik


> On Apr 25, 2017, at 1:46 PM, Ishan Chattopadhyaya  
> wrote:
> 
> Hi,
> We have lots of changes, optimizations, bug fixes for 6.6. I'd like to 
> propose a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0 
> release).
> 
> I can volunteer to release this, and I can cut the release branch around 4th 
> May, in order to let some time for devs to finish current issues.
> 
> What do you think?
> 
> Regards,
> Ishan
> 


-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



Release 6.6

2017-04-25 Thread Ishan Chattopadhyaya
Hi,
We have lots of changes, optimizations, bug fixes for 6.6. I'd like to
propose a 6.6 release (perhaps the last 6x non-bug-fix release before 7.0
release).

I can volunteer to release this, and I can cut the release branch around
4th May, in order to let some time for devs to finish current issues.

What do you think?

Regards,
Ishan