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 [EMAIL PROTECTED]  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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to