Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Invalid
    • Affects Version/s: Lucene.Net 2.9.2
    • Fix Version/s: Lucene.Net 3.0.3
    • Component/s: Lucene.Net Contrib
    • Labels:
      None

      Description

      I'm using Lucene.Net along with snowball stemming to index text from a database. The class Lucene.Net.Analysis.Snowball.SnowballFilter uses the reflection API and the invoke method to call the stem methods of snowball. I have written a Snowball filter which creates a delegate and uses this delegate to stem the words afterwards. This approach improves the indexing speed of my indexing program by about 10%. I would be happy if you include this code into lucene.net.

        Activity

        Hide
        Prescott Nasser added a comment -

        Has anyone taken a look at this code lately? I'm moving it from the Fix Version/s: Lucene.Net 2.9.4 because it wasn't included in that - but perhaps we should include this in 3.0.3

        Show
        Prescott Nasser added a comment - Has anyone taken a look at this code lately? I'm moving it from the Fix Version/s: Lucene.Net 2.9.4 because it wasn't included in that - but perhaps we should include this in 3.0.3
        Hide
        Christopher Currens added a comment -

        I haven't looked at the code, but I'm adding it into 3.0.3 anyway. When someone has the time to look over it and test it, we can decide if we plan on including it or not. At first glance, it looks good, but I wouldn't want to sign off on it until we're sure it fits in with the changes of 3.0.3 and passes all tests.

        Show
        Christopher Currens added a comment - I haven't looked at the code, but I'm adding it into 3.0.3 anyway. When someone has the time to look over it and test it, we can decide if we plan on including it or not. At first glance, it looks good, but I wouldn't want to sign off on it until we're sure it fits in with the changes of 3.0.3 and passes all tests.
        Hide
        Christopher Currens added a comment -

        It's been so long since this patch was submitted (2009), that it's no longer needed. The new version of the SnowballFilter from 3.0.3 only uses reflection in the constructor to create the filter (as does the patch). It's too bad this didn't make it into 2.9.4, where it could have really been used.

        Show
        Christopher Currens added a comment - It's been so long since this patch was submitted (2009), that it's no longer needed. The new version of the SnowballFilter from 3.0.3 only uses reflection in the constructor to create the filter (as does the patch). It's too bad this didn't make it into 2.9.4, where it could have really been used.

          People

          • Assignee:
            Unassigned
            Reporter:
            Arian Bär
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Due:
              Created:
              Updated:
              Resolved:

              Development