[
https://issues.apache.org/jira/browse/LUCENE-6759?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14723853#comment-14723853
]
Karl Wright edited comment on LUCENE-6759 at 8/31/15 6:55 PM:
--------------------------------------------------------------
So what I think may be happening is that when the bounds of the shape are
greater than the "size of the world", even just by a little bit, the recursion
does not continue (in that dimension, at least). Basically I think the code
simply doesn't expect a starting cell that large. There's no way, remember, to
even represent such a condition with integers, so the decision is a perfectly
reasonable one, but it's wrong.
I can try to demonstrate this with appropriate output. But I've already
demonstrated that geo3d is doing the right thing; it has the proper
relationships between all objects and between the point and the shape and the
area. So it *has* to be decision-making in the bkd descent that is problematic.
As for whether making FUDGE_FACTOR impact the docvalues encoding -- I agree
this is not ideal. I presented the patch solely because it worked.
was (Author: [email protected]):
So what I think may be happening is that when the bounds of the shape are
greater than the "size of the world", even just by a little bit, the recursion
does not continue (in that dimension, at least). Basically I think the code
simply doesn't expect a starting cell that large. There's no way, remember, to
even represent such a condition with integers, so the decision is a perfectly
reasonable one, but it's wrong.
I can try to demonstrate this with appropriate output. But I've already
demonstrated that geo3d is doing the right thing; it has the proper
relationships between all objects and between the point and the shape and the
area. So it *has* to be decision-making in the bkd descent that is problematic.
> Integrate lat/long BKD and spatial 3d, part 2
> ---------------------------------------------
>
> Key: LUCENE-6759
> URL: https://issues.apache.org/jira/browse/LUCENE-6759
> Project: Lucene - Core
> Issue Type: New Feature
> Reporter: Michael McCandless
> Attachments: LUCENE-6699.patch, LUCENE-6699.patch, LUCENE-6699.patch,
> LUCENE-6699.patch, LUCENE-6699.patch, LUCENE-6699.patch, LUCENE-6699.patch,
> LUCENE-6699.patch, LUCENE-6699.patch, LUCENE-6699.patch, LUCENE-6699.patch,
> LUCENE-6699.patch, LUCENE-6699.patch
>
>
> This is just a continuation of LUCENE-6699, which became too big.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
