Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-9972

SpellCheckComponent to return collations and suggestions as a JSON object rather than a list

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 6.5, 6.6
    • Response Writers
    • None

    Description

      original title: JSON-Specific Parameters arrntv causing some error for spellcheck component

      So I tried using the new array named list arrntv from solr 6.5 jenkins build
      but the json returned was broken when it returned response for spellcheck with word break.
      for example :

       {"name":"collation",{
              "type":"str","value":"collationQuery":"indomie kuing",
              "hits":81,
              "misspellingsAndCorrections":
              [
                {"name":"indomee","type":"str","value":"indomie"},
                {"name":"kuih","type":"str","value":"kuing"}
              ]}
       }
      

      as you may see that "collationQuery":"indomie kuing" was considered as value thus causing the json to fail.
      i think the correct json was :

      {"name":"collation",
              "type":"object",
              "value":{
              "collationQuery":"indomie kuing",
              "hits":81,
              "misspellingsAndCorrections":
              [
                {"name":"indomee","type":"str","value":"indomie"},
                {"name":"kuih","type":"str","value":"kuing"}
              ]}
       }
      

      sorry for bad grammar english was not my first language and i know that object was not supported by current arrntv options.

      Attachments

        1. SOLR-9972.patch
          13 kB
          Christine Poerschke
        2. SOLR-9972-hunch-no-test.patch
          1 kB
          Christine Poerschke
        3. SOLR-9972-impact
          3 kB
          Christine Poerschke
        4. SOLR-9972-impact.out
          18 kB
          Christine Poerschke
        5. SOLR-9972-with-test-after-SOLR-9975.patch
          4 kB
          Christine Poerschke

        Issue Links

          Activity

            People

              jdyer James Dyer
              dragonyui Ricky Oktavianus Lazuardy
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: