lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael McCandless (JIRA)" <>
Subject [jira] Commented: (LUCENE-1473) Implement Externalizable in main top level searcher classes
Date Tue, 02 Dec 2008 22:22:44 GMT


Michael McCandless commented on LUCENE-1473:

Do we really need to write the serialVersionUID?  That's adding 8 bytes to the storage of
each term.

The term storage is not particularly efficient when storing many terms in the same field,
because eg the String field is not written as an intern'd string.

Also I see many tests failing with this, eg TestBoolean2 -- I think we'll have to add:

   public Term() {}

so deserialization can work?  Which is then sort of annoying because it means it's possible
to create a Term with null field & text (though, you can do that anyway by passing in
"null" for each).

> Implement Externalizable in main top level searcher classes
> -----------------------------------------------------------
>                 Key: LUCENE-1473
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Search
>    Affects Versions: 2.4
>            Reporter: Jason Rutherglen
>            Priority: Minor
>         Attachments: LUCENE-1473.patch
> To maintain serialization compatibility between Lucene versions, major classes can implement
Externalizable.  This will make Serialization faster due to no reflection required and maintain
backwards compatibility.  

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