lucenenet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Shad Storhaug (JIRA)" <>
Subject [jira] [Closed] (LUCENENET-531) Incorrect parsing of when resetting a prefix tree node ending in LEAF_BYTE
Date Wed, 28 Jun 2017 18:16:00 GMT


Shad Storhaug closed LUCENENET-531.
    Resolution: Cannot Reproduce

The patch was committed to the main repo (
but it was never released since the last version of Lucene.Net.Contrib.Spatial was released
more than 6 months before this commit.

However, Lucene.Net.Contrib.Spatial is now dead. The new Lucene.Net.Spatial functionality
doesn't have a Node class - it has been replaced by a Cell class. Furthermore, in that class
Reset() always sets the token to null, so this fix would not make sense to do.

Unfortunately, this patch is missing the test we need to verify this problem is indeed gone
in Lucene.Net 4.8.0, but all existing Spatial tests ported from Lucene 4.8.0 pass. Feel free
to reopen or open a new issue/PR if you want to contribute a test or two to verify fix.

> Incorrect parsing of when resetting a prefix tree node ending in LEAF_BYTE
> --------------------------------------------------------------------------
>                 Key: LUCENENET-531
>                 URL:
>             Project: Lucene.Net
>          Issue Type: Bug
>          Components: Lucene.Net Contrib
>    Affects Versions: Lucene.Net 3.0.3
>         Environment: Lucene.Net.Contrib.Spatial
>            Reporter: John Diss
>            Priority: Critical
>         Attachments:
> When a prefix tree node is constructed, there is an explicit check to test if the last
char is LEAF_BYTE and its presence is handled appropriately. If the node is reused via the
node.Reset method there is no such check which leads to issues if presented with a token ending
in LEAF_BYTE. This manifests as "Unexpected char: +" in method QuadPrefixTree.QuadCell.MakeShape.
> I think this happens when building a cell from e.g a rectangle shape query parameter
where the cell level is < grid depth. 

This message was sent by Atlassian JIRA

View raw message