lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yonik Seeley (JIRA)" <>
Subject [jira] Commented: (LUCENE-2778) Allow easy extension of RAMDirectory
Date Fri, 26 Nov 2010 16:15:13 GMT


Yonik Seeley commented on LUCENE-2778:

Why was this committed to 3x and then merged to trunk?
Could we try to not merge to trunk and instead merge to 3x?   We're destroying the usefulness
of our history.
I thought Tokenizer just changed in trunk... I went to look at the history and the last 17
revisions are all identical (except for mergeprops I assume).  And I can't tell at a glance
what are real vs fake changes in the history.

> Allow easy extension of RAMDirectory
> ------------------------------------
>                 Key: LUCENE-2778
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Store
>            Reporter: Shai Erera
>            Assignee: Shai Erera
>            Priority: Minor
>             Fix For: 3.1, 4.0
>         Attachments: LUCENE-2778.patch
> RAMDirectory uses RAMFiles to store the data. RAMFile offers a newBuffer() method for
extensions to override and allocate buffers differently, from e.g. a pool or something. However,
RAMDirectory always allocates RAMFile and doesn't allow allocating a RAMFile extension, which
makes RAMFile.newBuffer() unusable.
> I think we can simply introduce a newRAMFile() method on RAMDirectory and make the RAMFiles
map protected, and it will allow really extending RAMDir.
> I will post a patch later.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message