rnewson commented on code in PR #5204: URL: https://github.com/apache/couchdb/pull/5204#discussion_r1729786177
########## src/docs/src/whatsnew/3.4.rst: ########## @@ -39,6 +39,14 @@ Breaking Changes We recommend adding ``nbf`` as a required claim if you know your tokens will include it. +* :ghissue:`5203`: Continuous change feeds with ``descending=true&limit=N`` + + Previously, ``_changes?feed=continuous&descending=true&limit=N`` with ``N`` + greater than total rows would block and wait on database changes. On each + change the first few sequence rows (sequences ``1-...``, ``2-...``) were + repeatedly emitted. The new behavior is to return after all the rows are Review Comment: The description of the new behaviour needs some work. will ponder over the weekend. We want to convey that the (internal) phase 2 of live updates will not be run (as they obviously can't be in the requested order). That is, we recognise that feed=continuous and descending=true are incompatible, and have chosen to honour the second setting (all the rows returned in the response are in descending order). as we know that any new update would belong at the beginning of the response, which is now long past, we choose to end the response. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
