Re: draft-ietf-imapext-condstore-05

[Ken Murchison <[e-mail]>]

Alexey Melnikov wrote:
> Ken Murchison wrote:
> 
>> Ken Murchison wrote:
>>
>>> I just starting working on a CONDSTORE implementation in Cyrus, and I 
>>> wanted to make a few comments/suggestions before the document gets 
>>> too far along in the queue.
>>>
>>> Here is what I have come up with over the last day.  I might have 
>>> more as I get further into my implementation.
>>
>>
>>
>> Section 3.6:  The phrase in the first sentence "to include 
>> mod-sequence" should be changed to read "to include the highest 
>> mod-sequence". 
> 
> 
> Correct (as per your explanation below)
> 
>> Since the mod-sequence being returned in the untagged SEARCH, SORT and 
>> possibly THREAD responses is the *highest* of the messages in the 
>> response, shouldn't the extended response tag be HIGHESTMODSEQ instead 
>> of MODSEQ?  This seems more intuitive, unless you think it conflicts 
>> with the HIGHESTMODSEQ response code. 
> 
> 
> I would like not to change syntax at this point. Once again, the draft 
> is already approved as RFC.

Right, I understand.  I'd also like to see SEARCH MODSEQ changed to 
SEARCH CHANGEDSINCE if we had the time.

-- 
Kenneth Murchison     Oceana Matrix Ltd.
Software Engineer     21 Princeton Place
716-662-8973 x26      Orchard Park, NY 14127
--PGP Public Key--    http://www.oceana.com/~ken/ksm.pgp