lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 31149] - [PATCH] to store binary fields with compression
Date Wed, 15 Sep 2004 16:04:13 GMT
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?

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).

With your current implementation a field that is indexed has to be dublicated
if it is stored in compressed form.

regards,
Christoph

---------------------------------------------------------------------
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