Re: [lucene] 02/03: Fix longstanding bug in path bounds calculation, and hook up efficient isWithin() and distance logic

2022-11-20 Thread Dawid Weiss
If you need a temporary commit to make the random tests always pass while I > diagnose and fix please let me know. > Please do, thanks. Dawid

Re: [lucene] 02/03: Fix longstanding bug in path bounds calculation, and hook up efficient isWithin() and distance logic

2022-11-19 Thread Karl Wright
These are randomized test failures occurring because a getBounds() operation is apparently not always returning the right thing and the new code depends on it being right. I can make a commit that disables this logic if it's annoying but the failures have been there all along. But now the code

Re: [lucene] 02/03: Fix longstanding bug in path bounds calculation, and hook up efficient isWithin() and distance logic

2022-11-19 Thread Karl Wright
I'm looking at it. Karl On Sat, Nov 19, 2022 at 11:41 PM Robert Muir wrote: > Multiple spatial tests are failing in jenkins... bisected them to this > commit. > > Can you please look into it? https://github.com/apache/lucene/issues/11956 > > On Sat, Nov 19, 2022 at 8:22 PM wrote: > > > > This

Re: [lucene] 02/03: Fix longstanding bug in path bounds calculation, and hook up efficient isWithin() and distance logic

2022-11-19 Thread Robert Muir
Multiple spatial tests are failing in jenkins... bisected them to this commit. Can you please look into it? https://github.com/apache/lucene/issues/11956 On Sat, Nov 19, 2022 at 8:22 PM wrote: > > This is an automated email from the ASF dual-hosted git repository. > > kwright pushed a commit to