Lucene - Core
  1. Lucene - Core
  2. LUCENE-5477

add near-real-time suggest building to AnalyzingInfixSuggester

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.8, Trunk
    • Component/s: modules/spellchecker
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      Because this suggester impl. is just a Lucene index under-the-hood, it should be straightforward to enable near-real-time additions/removals of suggestions.

      1. LUCENE-5477.patch
        38 kB
        Michael McCandless
      2. LUCENE-5477.patch
        54 kB
        Michael McCandless

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1d 5h 21m 1 Michael McCandless 28/Feb/14 16:55
        Resolved Resolved Closed Closed
        58d 6h 30m 1 Uwe Schindler 28/Apr/14 00:26
        Uwe Schindler made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Uwe Schindler added a comment -

        Close issue after release of 4.8.0

        Show
        Uwe Schindler added a comment - Close issue after release of 4.8.0
        Hide
        ASF subversion and git services added a comment -

        Commit 1573080 from Michael McCandless in branch 'dev/branches/lucene5376'
        [ https://svn.apache.org/r1573080 ]

        LUCENE-5376, LUCENE-5477: add near-real-time suggest updates when using AnalyzingInfixSuggester to Lucene demo server

        Show
        ASF subversion and git services added a comment - Commit 1573080 from Michael McCandless in branch 'dev/branches/lucene5376' [ https://svn.apache.org/r1573080 ] LUCENE-5376 , LUCENE-5477 : add near-real-time suggest updates when using AnalyzingInfixSuggester to Lucene demo server
        Michael McCandless made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Michael McCandless [ mikemccand ]
        Resolution Fixed [ 1 ]
        Hide
        ASF subversion and git services added a comment -

        Commit 1572997 from Michael McCandless in branch 'dev/branches/branch_4x'
        [ https://svn.apache.org/r1572997 ]

        LUCENE-5477: add near-real-time add/update/refresh to AnalyzingInfixSuggester

        Show
        ASF subversion and git services added a comment - Commit 1572997 from Michael McCandless in branch 'dev/branches/branch_4x' [ https://svn.apache.org/r1572997 ] LUCENE-5477 : add near-real-time add/update/refresh to AnalyzingInfixSuggester
        Hide
        ASF subversion and git services added a comment -

        Commit 1572992 from Michael McCandless in branch 'dev/trunk'
        [ https://svn.apache.org/r1572992 ]

        LUCENE-5477: add near-real-time add/update/refresh to AnalyzingInfixSuggester

        Show
        ASF subversion and git services added a comment - Commit 1572992 from Michael McCandless in branch 'dev/trunk' [ https://svn.apache.org/r1572992 ] LUCENE-5477 : add near-real-time add/update/refresh to AnalyzingInfixSuggester
        Michael McCandless made changes -
        Attachment LUCENE-5477.patch [ 12631723 ]
        Hide
        Michael McCandless added a comment -

        New patch, adding random test, which seems to be passing ... I think it's ready; I'll commit soon.

        Show
        Michael McCandless added a comment - New patch, adding random test, which seems to be passing ... I think it's ready; I'll commit soon.
        Hide
        Areek Zillur added a comment -

        Wow, that looks awesome! Thanks for getting rid of the redundant casting of InputIterator too.

        Show
        Areek Zillur added a comment - Wow, that looks awesome! Thanks for getting rid of the redundant casting of InputIterator too.
        Hide
        Robert Muir added a comment -

        this looks great!

        Show
        Robert Muir added a comment - this looks great!
        Michael McCandless made changes -
        Field Original Value New Value
        Attachment LUCENE-5477.patch [ 12631582 ]
        Hide
        Michael McCandless added a comment -

        Initial patch, with new add/update/refresh methods added to AnalyzingInfixSuggester.

        I added a "testBasicNRT" and it seems to pass, but I still need to add a randomized test.

        I think the approach will work well: I'm just using SortingMergePolicy and EarlyTerminatingSortingCollector, and I switched to SearcherManager to pull the current searcher.

        Show
        Michael McCandless added a comment - Initial patch, with new add/update/refresh methods added to AnalyzingInfixSuggester. I added a "testBasicNRT" and it seems to pass, but I still need to add a randomized test. I think the approach will work well: I'm just using SortingMergePolicy and EarlyTerminatingSortingCollector, and I switched to SearcherManager to pull the current searcher.
        Michael McCandless created issue -

          People

          • Assignee:
            Michael McCandless
            Reporter:
            Michael McCandless
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development