rnewson commented on PR #6138: URL: https://github.com/apache/couchdb/pull/6138#issuecomment-5893218995
awesome, will review soon! My broader plan is to remove the update-time sequence checking entirely. That was added to address a problem that cannot happen since I added the bulk update endpoint. I'd remove the individual doc delete/update/purge endpoints entirely too. In that future version the index update process is easier to follow and is what you've surmised from the code; 1) apply all the purges (but don't set sequences on the index) 2) then apply the changes feed updates 3) then set update_seq and purge_seq to that of the db snapshot we grabbed at the start of the update the fact that steps 1 and 2 proceed by building a batch of updates and making a bulk update request to nouveau is a performance detail (that also happens to ensure the order of index updates as the request body is an ordered list). -- 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]
