davisp commented on issue #2666: soft-deletion for database URL: https://github.com/apache/couchdb/pull/2666#issuecomment-607973871 Also, something I noticed while playing with the tests is that the `GET /_deleted_dbs/DbName` is not very ergonomic. Due to how things are implemented, if we don't have any deleted snapshots we still return a `200` response with an empty array `[]` as the body. I would have expected a `404` in that situation given that the `DbName` is part of the URL. I realize that's purely due to how things are implemented, but it suggests to me that the `GET /_deleted_dbs?key=DbName` approach is better since we'd be more likely to understand that a `200` and empty array returned is expected since its more like a view query where results might or might not exist.
---------------------------------------------------------------- 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] With regards, Apache Git Services
