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

SpellcheckComponent throws NullPointerException on restart without build

        Details

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

          Description

          from http://mail-archives.apache.org/mod_mbox/lucene-solr-user/200807.mbox/%3cC4AC838E.ADE7%25jonathan_lee@comcast.com%3e

          "I believe there is a bug in IndexBased- and FileBasedSpellChecker.java
          where the analyzer variable is only set on the build command. Therefore,
          when the index is reloaded, but not built after starting solr, issuing a
          query with the spellcheck.q parameter will cause a NullPointerException to
          be thrown (SpellCheckComponent.java:158). Moving the analyzer logic to the
          constructor seems to fix the problem."

          1. Text File
            SOLR-648.patch
            16 kB
            Shalin Shekhar Mangar
          2. Text File
            SOLR-648.patch
            13 kB
            Shalin Shekhar Mangar
          3. Text File
            SOLR-648.patch
            11 kB
            Jonathan Lee

            Issue Links

            blocks

            Improvement - An improvement or enhancement to an existing feature or task. SOLR-622 SpellCheckComponent should build or load indices on startup and commits

            • Minor - Minor loss of function, or other problem where easy workaround is present.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              jonjlee Jonathan Lee added a comment -

              Here is the patch that was originally sent by email, which probably needs to be changed: http://mail-archives.apache.org/mod_mbox/lucene-solr-user/200807.mbox/%3cC4ACB4E4.AE1F%25jonathan_lee@comcast.com%3e

              Show
              jonjlee Jonathan Lee added a comment - Here is the patch that was originally sent by email, which probably needs to be changed: http://mail-archives.apache.org/mod_mbox/lucene-solr-user/200807.mbox/%3cC4ACB4E4.AE1F%25jonathan_lee@comcast.com%3e
              Hide
              Permalink
              shalinmangar Shalin Shekhar Mangar added a comment -

              This fixes the NullPointerException. The changes are:

              • The signature of the init method is changed to init(NamedList, SolrCore)
              • All analyzer defining code is moved to AbstractLuceneSpellChecker#init from the build methods in IndexBasedSpellChecker and FileBasedSpellChecker.

              I'll add a test and then give another patch shortly.

              Show
              shalinmangar Shalin Shekhar Mangar added a comment - This fixes the NullPointerException. The changes are: The signature of the init method is changed to init(NamedList, SolrCore) All analyzer defining code is moved to AbstractLuceneSpellChecker#init from the build methods in IndexBasedSpellChecker and FileBasedSpellChecker. I'll add a test and then give another patch shortly.
              Hide
              Permalink
              shalinmangar Shalin Shekhar Mangar added a comment -

              Added testcase as SpellCheckComponentTest#testReloadOnStart which reproduces the bug reported in this issue.

              Grant, would you like to review this patch? It contains minor API change so you'll need to revisit the custom SolrSpellChecker implementation you had. I'd like to commit this asap so that I can continue with SOLR-622.

              An alternate way would have been to change the signature of the SolrSpellChecker.reload method to take in atleast the SolrCore or the same params as the build method.

              Show
              shalinmangar Shalin Shekhar Mangar added a comment - Added testcase as SpellCheckComponentTest#testReloadOnStart which reproduces the bug reported in this issue. Grant, would you like to review this patch? It contains minor API change so you'll need to revisit the custom SolrSpellChecker implementation you had. I'd like to commit this asap so that I can continue with SOLR-622 . An alternate way would have been to change the signature of the SolrSpellChecker.reload method to take in atleast the SolrCore or the same params as the build method.
              Hide
              Permalink
              gsingers Grant Ingersoll added a comment -

              I perused the patch and it looks reasonable since we are just getting the ResourceLoader from the Core there anyway. It does look like we should add a test case for this issue.

              Show
              gsingers Grant Ingersoll added a comment - I perused the patch and it looks reasonable since we are just getting the ResourceLoader from the Core there anyway. It does look like we should add a test case for this issue.
              Hide
              Permalink
              shalinmangar Shalin Shekhar Mangar added a comment -

              Yes, I've added a testcase which fails otherwise.

              Show
              shalinmangar Shalin Shekhar Mangar added a comment - Yes, I've added a testcase which fails otherwise.
              Hide
              Permalink
              shalinmangar Shalin Shekhar Mangar added a comment - - edited

              Committed revision 679816.

              Thanks Jonathan and Geoffrey!

              Show
              shalinmangar Shalin Shekhar Mangar added a comment - - edited Committed revision 679816. Thanks Jonathan and Geoffrey!

                People

                • Assignee:
                  shalinmangar Shalin Shekhar Mangar
                  Reporter:
                  geoff Geoffrey Young
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development