[jira] Commented: (LUCENE-743) IndexReader.reopen()

2007-07-20 Thread Michael Busch (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-743?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12514131 ] Michael Busch commented on LUCENE-743: -- > i somehow missed seeing this issues before ... actually, me too... f

[jira] Updated: (LUCENE-743) IndexReader.reopen()

2007-07-20 Thread Michael Busch (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-743?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael Busch updated LUCENE-743: - Attachment: lucene-743.patch First version of my patch: * Adds the static method IndexReader.

Re: Token termBuffer issues

2007-07-20 Thread Yonik Seeley
On 7/19/07, Michael McCandless <[EMAIL PROTECTED]> wrote: "Yonik Seeley" <[EMAIL PROTECTED]> wrote: > I had previously missed the changes to Token that add support for > using an array (termBuffer): > > + // For better indexing speed, use termBuffer (and > + // termBufferOffset/termBufferLength

Re: Token termBuffer issues

2007-07-20 Thread Chris Hostetter
: > Currently the char[] wins, but good point: seems like each setter : > should null out the other one? : : Certainly the String setter should null the char[] (that's the only : way to keep back compatibility), and probably vice-versa. i haven't really thought baout this before today (i missed s

[jira] Commented: (LUCENE-743) IndexReader.reopen()

2007-07-20 Thread Hoss Man (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-743?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12514379 ] Hoss Man commented on LUCENE-743: - > Yeah we could do that. However, this might not be so easy to implement. > For e

[jira] Updated: (LUCENE-743) IndexReader.reopen()

2007-07-20 Thread Hoss Man (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-743?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Hoss Man updated LUCENE-743: Attachment: lucene-743.patch an extremely hackish refactoring of the previous patch that demonstrates the