lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Grant Ingersoll <gsing...@apache.org>
Subject Re: Payload API
Date Sun, 18 Nov 2007 13:54:37 GMT
+1 for the signature changes and +1 for adding the field name.


-Grant

On Nov 18, 2007, at 6:07 AM, Michael McCandless wrote:

>
> "Yonik Seeley" <yonik@apache.org> wrote:
>> So I think we should change + finalize the payload API before Lucene
>> 2.3 comes out.
>>
>> Single biggest drawback about current payloads is that there isn't  
>> any
>> explicit support for adding different types of payloads to the same
>> token.
>> I don't really see a good fix to that though, so I'm OK with leaving
>> that as it is.
>>
>> One change I'd support is to remove the Payload class and replace it
>> with members directly on Token:
>>
>> class Token {
>>    byte[] payload;
>>    int payloadLength;
>>    void setPayload(byte[], int length)
>>    byte[] getPayload()
>>    int getPayloadLength()
>> ...
>> }
>
> +1
>
> Mike
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org
> For additional commands, e-mail: java-dev-help@lucene.apache.org
>



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


Mime
View raw message