[GitHub] [couchdb-mango] garrensmith commented on pull request #48: adds operators $sizeLte and $sizeGte

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

garrensmith commented on pull request #48:
URL: https://github.com/apache/couchdb-mango/pull/48#issuecomment-847790726


   Hi @hklarner this is an interesting idea and thank you for adding the code. A few things will have to be done before this could land:
   
   * We no longer use this repo. Mango is fully integrated into the main couchdb repo https://github.com/apache/couchdb/tree/main/src/mango so you will have to apply this patch to the couchdb repo
   * Could you add some tests to check this works as expected and handles the error cases
   * Where did you get the `$sizeLte` and `$sizeGte` operators. I couldn't find them in Mongodb. So the idea sounds good but I think you might have to send an email to the Couchdb dev mailing saying you want to add these operators and check everyone is happy with that. 


-- 
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