[GitHub] [couchdb] rnewson commented on pull request #3038: feat: per-document-access

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

rnewson commented on pull request #3038:
URL: https://github.com/apache/couchdb/pull/3038#issuecomment-679149829


   another note. In some places you use a list comprehension, in others a lists:map. In both cases you're changing each item of a list to superset or subset of its original content, but using two different mechanisms to do so. It wasn't clear if this was intentional or not. The only difference is error handling, in that the list comprehension approach can silently drop items that fail, whereas lists:map would not. Is that why you chose lc vs map or were the choices arbitrary?


----------------------------------------------------------------
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:
users@infra.apache.org