Solr
  1. Solr
  2. SOLR-1558

QueryElevationComponent should fail to init if uniqueKey field isn't string

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3, 1.4
    • Fix Version/s: 1.4.1, 1.5, 3.1, 4.0-ALPHA
    • Component/s: search
    • Labels:
      None

      Description

      QueryElevationComponent fails in confusing ways if you use it on a schema where the uniqueKey fieldtype is not a StrField. This is easy to assert at init, so we should do that.

        Issue Links

          Activity

          Hide
          Hoss Man added a comment -

          SOLR-1520 request that we make QEC work with any uniqueKey type, so it is related – but that goal is likely harder to achieve then a simple/helpful error message on startup so i'm tracking them as separate issues.

          Show
          Hoss Man added a comment - SOLR-1520 request that we make QEC work with any uniqueKey type, so it is related – but that goal is likely harder to achieve then a simple/helpful error message on startup so i'm tracking them as separate issues.
          Hide
          Hoss Man added a comment -

          Committed revision 884393.

          Note that the code change only required modifying two lines, but this fix exposed 7 test config files that had cut/paste declaations of the elevation component that weren't used, and never would have worked because of the schema used in the tests.

          Show
          Hoss Man added a comment - Committed revision 884393. Note that the code change only required modifying two lines, but this fix exposed 7 test config files that had cut/paste declaations of the elevation component that weren't used, and never would have worked because of the schema used in the tests.
          Hide
          Hoss Man added a comment -

          Correcting Fix Version based on CHANGES.txt, see this thread for more details...

          http://mail-archives.apache.org/mod_mbox/lucene-dev/201005.mbox/%3Calpine.DEB.1.10.1005251052040.24672@radix.cryptio.net%3E

          Show
          Hoss Man added a comment - Correcting Fix Version based on CHANGES.txt, see this thread for more details... http://mail-archives.apache.org/mod_mbox/lucene-dev/201005.mbox/%3Calpine.DEB.1.10.1005251052040.24672@radix.cryptio.net%3E
          Hide
          Hoss Man added a comment -

          Committed revision 949888.

          merged to branch-1.4 for 1.4.1

          Show
          Hoss Man added a comment - Committed revision 949888. merged to branch-1.4 for 1.4.1

            People

            • Assignee:
              Hoss Man
              Reporter:
              Hoss Man
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development