Details

    • Type: Test Test
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.3, 6.0
    • Component/s: None
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      Patch to expose MappedByteBuffer.load() as a simple boolean option.

      This can be used as an alternative to copy-into-RAMDirectory.

      Users who want sophisticated logic about this can use FileSwitchDirectory or other methods.

        Activity

        Hide
        Uwe Schindler added a comment -

        Hi,
        the patch is OK. But I have no strong opinion about this - I just would not use it, but anybody has the option now, so +1

        We should just not make this the default or suggest anywhere to turn it on by default! The problem si: You have to load the data anywaysand I would prefer to do it as needed for faster startup time. It is also very bad for users who have not enough physical RAM, because it would load the stuff, invalidate a lot of active cache pages and then other stuff gets slower. I think the O/S does good job in paging, sorry! A short delay for some queries returning many results is not a problem for me (my personal opinion). But invalidating cache pages of more important stuff is bad.

        Show
        Uwe Schindler added a comment - Hi, the patch is OK. But I have no strong opinion about this - I just would not use it, but anybody has the option now, so +1 We should just not make this the default or suggest anywhere to turn it on by default! The problem si: You have to load the data anywaysand I would prefer to do it as needed for faster startup time. It is also very bad for users who have not enough physical RAM, because it would load the stuff, invalidate a lot of active cache pages and then other stuff gets slower. I think the O/S does good job in paging, sorry! A short delay for some queries returning many results is not a problem for me (my personal opinion). But invalidating cache pages of more important stuff is bad.
        Hide
        ASF subversion and git services added a comment -

        Commit 1684968 from Robert Muir in branch 'dev/trunk'
        [ https://svn.apache.org/r1684968 ]

        LUCENE-6549: add preload option to MMapDirectory

        Show
        ASF subversion and git services added a comment - Commit 1684968 from Robert Muir in branch 'dev/trunk' [ https://svn.apache.org/r1684968 ] LUCENE-6549 : add preload option to MMapDirectory
        Hide
        ASF subversion and git services added a comment -

        Commit 1684974 from Robert Muir in branch 'dev/branches/branch_5x'
        [ https://svn.apache.org/r1684974 ]

        LUCENE-6549: add preload option to MMapDirectory

        Show
        ASF subversion and git services added a comment - Commit 1684974 from Robert Muir in branch 'dev/branches/branch_5x' [ https://svn.apache.org/r1684974 ] LUCENE-6549 : add preload option to MMapDirectory
        Hide
        Shalin Shekhar Mangar added a comment -

        Bulk close for 5.3.0 release

        Show
        Shalin Shekhar Mangar added a comment - Bulk close for 5.3.0 release

          People

          • Assignee:
            Unassigned
            Reporter:
            Robert Muir
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development