Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 6.1, 6.0.1, 7.0
    • Component/s: None
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      We had this for e.g. docvalues fields from the beginning:

       * If you also need to store the value, you should add a
       * separate {@link StoredField} instance.
      

      We should add this to the points types too? It will prevent confusion.

      1. LUCENE-7223.patch
        4 kB
        Michael McCandless

        Activity

        Hide
        mikemccand Michael McCandless added a comment -

        +1, I'll do this.

        Show
        mikemccand Michael McCandless added a comment - +1, I'll do this.
        Hide
        mikemccand Michael McCandless added a comment -

        Simple patch, just fixing the core XXXPoint classes. I was going to also update sandbox and spatial3d but it's less simple there since StoredField can't directly store e.g. a BigInteger.

        Show
        mikemccand Michael McCandless added a comment - Simple patch, just fixing the core XXXPoint classes. I was going to also update sandbox and spatial3d but it's less simple there since StoredField can't directly store e.g. a BigInteger .
        Hide
        rcmuir Robert Muir added a comment -

        Right, we dont have predefined "storage" in StoredField for all possible types. Nor do we for even a 2 dimensional IntPoint.

        But the user can always store this as binary/string/however they want for these cases.

        +1 for the patch!

        Show
        rcmuir Robert Muir added a comment - Right, we dont have predefined "storage" in StoredField for all possible types. Nor do we for even a 2 dimensional IntPoint. But the user can always store this as binary/string/however they want for these cases. +1 for the patch!
        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit c7d4080820b0b50d09c7e1ddb5e68b325ab145b7 in lucene-solr's branch refs/heads/master from Mike McCandless
        [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=c7d4080 ]

        LUCENE-7223: improve XXXPoint javadocs to explain that a StoredField is needed if you want to retrieve the original value at search time

        Show
        jira-bot ASF subversion and git services added a comment - Commit c7d4080820b0b50d09c7e1ddb5e68b325ab145b7 in lucene-solr's branch refs/heads/master from Mike McCandless [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=c7d4080 ] LUCENE-7223 : improve XXXPoint javadocs to explain that a StoredField is needed if you want to retrieve the original value at search time
        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit 8f2a3a52713a0c170fd19e432e48c04c77e37654 in lucene-solr's branch refs/heads/branch_6x from Mike McCandless
        [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=8f2a3a5 ]

        LUCENE-7223: improve XXXPoint javadocs to explain that a StoredField is needed if you want to retrieve the original value at search time

        Show
        jira-bot ASF subversion and git services added a comment - Commit 8f2a3a52713a0c170fd19e432e48c04c77e37654 in lucene-solr's branch refs/heads/branch_6x from Mike McCandless [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=8f2a3a5 ] LUCENE-7223 : improve XXXPoint javadocs to explain that a StoredField is needed if you want to retrieve the original value at search time
        Hide
        hossman Hoss Man added a comment -

        Manually correcting fixVersion per Step #S5 of LUCENE-7271

        Show
        hossman Hoss Man added a comment - Manually correcting fixVersion per Step #S5 of LUCENE-7271
        Hide
        steve_rowe Steve Rowe added a comment -

        Reopening to backport to 6.0.1.

        Show
        steve_rowe Steve Rowe added a comment - Reopening to backport to 6.0.1.
        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit 3e77726b88efad4bcb2127ee6ce393f20dff9f4e in lucene-solr's branch refs/heads/branch_6_0 from Mike McCandless
        [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=3e77726 ]

        LUCENE-7223: improve XXXPoint javadocs to explain that a StoredField is needed if you want to retrieve the original value at search time

        Show
        jira-bot ASF subversion and git services added a comment - Commit 3e77726b88efad4bcb2127ee6ce393f20dff9f4e in lucene-solr's branch refs/heads/branch_6_0 from Mike McCandless [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=3e77726 ] LUCENE-7223 : improve XXXPoint javadocs to explain that a StoredField is needed if you want to retrieve the original value at search time
        Hide
        steve_rowe Steve Rowe added a comment -

        Bulk close issues included in the 6.0.1 release.

        Show
        steve_rowe Steve Rowe added a comment - Bulk close issues included in the 6.0.1 release.

          People

          • Assignee:
            Unassigned
            Reporter:
            rcmuir Robert Muir
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development