Solr
  1. Solr
  2. SOLR-3211

Allow parameter override in conjunction with "spellcheck.maxCollationTries"

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.6, 4.0-ALPHA
    • Fix Version/s: 4.0-ALPHA, 5.0
    • Component/s: spellchecker
    • Labels:
      None

      Description

      A couple users on the mailing list recently asked about being able to override the "mm" parameter when SpellCheckComponent issues queries to check for # hits for a collation candidate. The issue is if the query had mm=0, pretty much everything will generate hits. But for collation checking purposes, a low "mm" is almost never desirable.

      It might be worthwhile to generalize this to let other parameters be overridden as well.

        Activity

        Hide
        James Dyer added a comment -

        Committed...Trunk: 1348936, Branch_4x: r1348937

        Show
        James Dyer added a comment - Committed...Trunk: 1348936, Branch_4x: r1348937
        Hide
        James Dyer added a comment -

        Here is a patch for this with a unit test.

        This creates a new parameter syntax: spellcheck.collateParam.xx where "xx" is the parameter you are overriding. Typically users would want to override "mm" from a low value to a high value so that the spellchecker is truly verifying that the replacement (collation) terms exist together in the same document. I made this more general in case users wanted to override something different (perhaps "q.op").

        I would like to commit this one in a few days or so...

        Show
        James Dyer added a comment - Here is a patch for this with a unit test. This creates a new parameter syntax: spellcheck.collateParam.xx where "xx" is the parameter you are overriding. Typically users would want to override "mm" from a low value to a high value so that the spellchecker is truly verifying that the replacement (collation) terms exist together in the same document. I made this more general in case users wanted to override something different (perhaps "q.op"). I would like to commit this one in a few days or so...

          People

          • Assignee:
            James Dyer
            Reporter:
            James Dyer
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development