lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Bernhard Messer <Bernhard.Mes...@intrafind.de>
Subject Re: DO NOT REPLY [Bug 31149] - [PATCH] to store binary fields with compression
Date Mon, 27 Sep 2004 16:01:42 GMT
bugzilla@apache.org wrote:

>DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
>RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
><http://issues.apache.org/bugzilla/show_bug.cgi?id=31149>.
>ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
>INSERTED IN THE BUG DATABASE.
>
>http://issues.apache.org/bugzilla/show_bug.cgi?id=31149
>
>[PATCH] to store binary fields with compression
>
>
>
>
>
>------- Additional Comments From goller@detego-software.de  2004-09-15 16:04 -------
>Hi Bernhard,
>
>I reviewed your patch. Looks great for me. However, I wonder why we need
>isCompressed in FieldInfo? Beeing compressed or not seems to be a property of an
>individual field more than of all fields in the index with a given name.
>Furthermore, the isCompressed flag in FieldInfo is currently not used anywhere
>outside FieldInfo and FieldInfos. Is it really needed?
>  
>
seems that you are right. I will check, if it is enough to store the 
information whether a field is compressed or not in the bit value only. 
Currently i have no idea why there was a change to FieldInfos necessary ;-)

>Further idea: Wouldn't it be great to have a stored stringValued field that
>has the property "compressed" meaning that if the field is written with
>FieldsWriter, it automatically is compressed and if it's read by FieldsReader,
>it is automatically decompressed and transformed into a String? The field could
>but does not have to be indexed/tokenized. This would mean that compressed
>becomes a property of stored fields (binary or stringValued ones).
>
>  
>
yes you're right. I'll give it a try and check how an implementation 
could be done.

>With your current implementation a field that is indexed has to be dublicated
>if it is stored in compressed form.
>
>  
>
If i got you're idea right, you want to have all fields which are 
stored, optionally compressed ?

thanks
bernhard

>regards,
>Christoph
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: lucene-dev-unsubscribe@jakarta.apache.org
>For additional commands, e-mail: lucene-dev-help@jakarta.apache.org
>
>  
>


---------------------------------------------------------------------
To unsubscribe, e-mail: lucene-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: lucene-dev-help@jakarta.apache.org


Mime
View raw message