Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.6, 6.0
    • Component/s: core/search
    • Labels:
      None
    • Lucene Fields:
      New, Patch Available

      Description

      It would be nice to have a sizeInBytes() method added to Suggester.Lookup interface. This would allow users to estimate the size of the in-memory data structure created by various suggester implementation.

        Activity

        Hide
        Areek Zillur added a comment -

        Initial patch implementing sizeInBytes() for Suggest.Lookup.

        Show
        Areek Zillur added a comment - Initial patch implementing sizeInBytes() for Suggest.Lookup.
        Hide
        Dawid Weiss added a comment -

        Looks good to me.

        Show
        Dawid Weiss added a comment - Looks good to me.
        Hide
        Michael McCandless added a comment -

        Looks great, I'll commit shortly. Thanks Areek!

        Show
        Michael McCandless added a comment - Looks great, I'll commit shortly. Thanks Areek!
        Hide
        ASF subversion and git services added a comment -

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

        LUCENE-5323: add Lookup.sizeInBytes

        Show
        ASF subversion and git services added a comment - Commit 1538578 from Michael McCandless in branch 'dev/trunk' [ https://svn.apache.org/r1538578 ] LUCENE-5323 : add Lookup.sizeInBytes
        Hide
        ASF subversion and git services added a comment -

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

        LUCENE-5323: add Lookup.sizeInBytes

        Show
        ASF subversion and git services added a comment - Commit 1538579 from Michael McCandless in branch 'dev/branches/branch_4x' [ https://svn.apache.org/r1538579 ] LUCENE-5323 : add Lookup.sizeInBytes
        Hide
        Areek Zillur added a comment -

        Thanks Dawid and Michael!

        Show
        Areek Zillur added a comment - Thanks Dawid and Michael!

          People

          • Assignee:
            Unassigned
            Reporter:
            Areek Zillur
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development