Anyone have any objections to committing this addition to Term.java?

_http://www.mail-archive.com/java-dev@lucene.apache.org/msg00618.html_

It's a simple addition to avoid fieldName.intern() overheads by safely constructing new Term objects from existing Term objects and re-using it's pre-interned field name. The alternative solution would be to make the existing Term(field, value, isInterned) constructor public with a suitable javadoc warning about the caller ensuring use pre-interned fieldnames.

The latest version of MemoryIndex is lingering in BugZilla waiting for the above change. I'd like to update the contrib area with this too if everyone is OK with this. The MemoryIndex's justification for the Term change is here _http://issues.apache.org/bugzilla/show_bug.cgi?id=34585_

Cheers
Mark


        
        
                
___________________________________________________________ Yahoo! Messenger - NEW crystal clear PC to PC calling worldwide with voicemail http://uk.messenger.yahoo.com

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to