Solr
  1. Solr
  2. SOLR-1771

StringIndexDocValues should provide a better error message when getStringIndex fails

    Details

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

      Description

      if someone attempts to use an OrdFieldSource on a field that is tokenized, FieldCache.getStringIndex throws a confusing RuntimeException that StringIndexDocValues propogates. we should wrap that exception in something more helpful...

      http://old.nabble.com/sorting-td27544348.html

        Activity

        Hide
        Hoss Man added a comment -

        I'm not convinced that the wording of the new error message is all that great, but it's vastly better then the previous behavior...

        Committed revision 909746.

        Note that this affected numerous different class: OrdFieldSource, All the "Sortable*Field" classes, DateField, and StrField. (anyone instantiating an instance of StringIndexDocValues)

        Show
        Hoss Man added a comment - I'm not convinced that the wording of the new error message is all that great, but it's vastly better then the previous behavior... Committed revision 909746. Note that this affected numerous different class: OrdFieldSource, All the "Sortable*Field" classes, DateField, and StrField. (anyone instantiating an instance of StringIndexDocValues)
        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
        Grant Ingersoll added a comment -

        Bulk close for 3.1.0 release

        Show
        Grant Ingersoll added a comment - Bulk close for 3.1.0 release

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development