Solr
  1. Solr
  2. SOLR-4230

Enhance geofilt and bbox parsers to support Solr 4 spatial field types

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.1, 6.0
    • Component/s: search
    • Labels:
      None

      Description

      CHANGES.txt entry:

      * SOLR-4230: The new Solr 4 spatial fields now work with the {!geofilt} and
        {!bbox} query parsers. The score local-param works too. (David Smiley)
      

        Activity

        Hide
        David Smiley added a comment -

        The attached file implements this support. Note that the score="distance" local-param feature in the Solr 4 spatial field should also work too – pretty cool.

        I took a look at adding support for the "geodist" function, which is implemented in HaversineConstFunction. Ugh, the logic therein is convoluted and it makes heavy assumptions about a "MultiValueSource" based field which the Solr 4 spatial fields don't use and never will. Adding support here for Solr 4 spatial fields would definitely be a hack and on top of confusing code that I don't want to make any more confusing. So I'll pass on that.

        I plan to commit in a few days.

        Show
        David Smiley added a comment - The attached file implements this support. Note that the score="distance" local-param feature in the Solr 4 spatial field should also work too – pretty cool. I took a look at adding support for the "geodist" function, which is implemented in HaversineConstFunction. Ugh, the logic therein is convoluted and it makes heavy assumptions about a "MultiValueSource" based field which the Solr 4 spatial fields don't use and never will. Adding support here for Solr 4 spatial fields would definitely be a hack and on top of confusing code that I don't want to make any more confusing. So I'll pass on that. I plan to commit in a few days.
        Hide
        Commit Tag Bot added a comment -

        [trunk commit] David Wayne Smiley
        http://svn.apache.org/viewvc?view=revision&revision=1428063

        SOLR-4230 geofilt and bbox support for Solr 4 spatial

        Show
        Commit Tag Bot added a comment - [trunk commit] David Wayne Smiley http://svn.apache.org/viewvc?view=revision&revision=1428063 SOLR-4230 geofilt and bbox support for Solr 4 spatial
        Hide
        Commit Tag Bot added a comment -

        [trunk commit] David Wayne Smiley
        http://svn.apache.org/viewvc?view=revision&revision=1428067

        SOLR-4230: CHANGES.txt

        Show
        Commit Tag Bot added a comment - [trunk commit] David Wayne Smiley http://svn.apache.org/viewvc?view=revision&revision=1428067 SOLR-4230 : CHANGES.txt
        Hide
        Commit Tag Bot added a comment -

        [branch_4x commit] David Wayne Smiley
        http://svn.apache.org/viewvc?view=revision&revision=1428068

        SOLR-4230 geofilt and bbox support for Solr 4 spatial

        Show
        Commit Tag Bot added a comment - [branch_4x commit] David Wayne Smiley http://svn.apache.org/viewvc?view=revision&revision=1428068 SOLR-4230 geofilt and bbox support for Solr 4 spatial

          People

          • Assignee:
            David Smiley
            Reporter:
            David Smiley
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development