[GitHub] [couchdb] tonysun83 edited a comment on pull request #3909: Fix new_edits:false and VDU function_clause

[GitBox <gi...@apache.org>]

tonysun83 edited a comment on pull request #3909:
URL: https://github.com/apache/couchdb/pull/3909#issuecomment-1020539556


   > In other words, we'd assert that there is at least one forbidden error (and all the rest of forbidden should match) and all the other responses are noreply.
   
   @nickva I'm trying to figure out in what situations where we wouldn't want to forbid the document even if the replies were not `noreply`. 
   
   ```
   [{_, {forbidden, _}, Reply2, Reply3]
   ```
   Reply2, and Reply3 can override forbidden only if they are equal to successful  `Reply1 = {ok, Doc1}`, `Reply2 = {ok, Doc2}`. That doesn't seem like it can happen because a successful update implies an update to the revision tree, and the VDU would prevent it.
   
   I can't think of any other scenarios where and Reply2 and Reply3 should override `forbidden`


-- 
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: notifications-unsubscribe@couchdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org