Solr
  1. Solr
  2. SOLR-1491

single valued trie fields use multi valued faceting

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.4
    • Fix Version/s: 1.4
    • Component/s: None
    • Labels:
      None

      Description

      Single-valued trie fields use multi-valued faceting because isTokenized returns true.
      Discovered here: http://search.lucidimagination.com/search/document/e00bcf02427d404b/solr_trunk_heap_space_issues

      1. SOLR-1491.patch
        3 kB
        Yonik Seeley

        Activity

        Hide
        Yonik Seeley added a comment -

        Patch attached... here's the essence of it:

          /** Returns true if a single field value of this type has multiple logical values
           *  for the purposes of faceting, sorting, etc.  Text fields normally return
           *  true since each token/word is a logical value.
           */
          public boolean multiValuedFieldCache() {
            return isTokenized();
          }
        

        I'm not crazy about the name/description, but I can't currently think of better.

        Show
        Yonik Seeley added a comment - Patch attached... here's the essence of it: /** Returns true if a single field value of this type has multiple logical values * for the purposes of faceting, sorting, etc. Text fields normally return * true since each token/word is a logical value. */ public boolean multiValuedFieldCache() { return isTokenized(); } I'm not crazy about the name/description, but I can't currently think of better.
        Hide
        Yonik Seeley added a comment -

        This was committed as part of SOLR-1492

        Show
        Yonik Seeley added a comment - This was committed as part of SOLR-1492
        Hide
        Grant Ingersoll added a comment -

        Bulk close for Solr 1.4

        Show
        Grant Ingersoll added a comment - Bulk close for Solr 1.4

          People

          • Assignee:
            Yonik Seeley
            Reporter:
            Yonik Seeley
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development