nickva edited a comment on pull request #3488:
URL: https://github.com/apache/couchdb/pull/3488#issuecomment-812723151


   I thought of making it bit more general since noticed we'd have to handle 
perhaps `less({[?MAX_JSON_OBJ], _}, _) -> gt` and maybe also `{[?MAX_JSON_OBJ], 
_}, {[?MAX_JSON_OBJ], _} -> eq`. The one clause would work currently as is,  
but, if we change the logic in the future it might get an unexpected failure. 
Then there is also the `?HIGHEST_KEY` macro which I think is not used in `main` 
but it may be one day before CentOS 7 is retired and it would unexpectedly fail 
as well.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to