> recordSetTTL  -> resultSetTTL  (two occurences I think)

OK
 
> 
> Thinko:
> 
> resultSetIdleTime should be positive Integer, not non-negative.
> (as you shouldn't return a result set that it would be impossible to
> interact with)

I thought a resultSetIdleTime of 0 was a way of the client "deleting" a
resultSet (or strongly hinting that it is no longer needed)


> code in diagnostics should be positivie Integer, as there is 
> no diagnostic
> 0?

OK

> 
> Thought:
> 
> nextRecordPosition should be positive as well, and should be 
> omitted if
> there is not another record, rather than returning 
> nextRecordPosition of
> 0? As nRP is there for terminally braindead clients, it's 
> easier for the
> server to omit it rather than make the client check if it's 0.

Makes sense.

Matthew