[ 
https://issues.apache.org/jira/browse/LUCENE-1473?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12652546#action_12652546
 ] 

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: https://issues.apache.org/jira/browse/LUCENE-1473
>             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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to