New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: COUNT Queries #1774
feat: COUNT Queries #1774
Conversation
3605570
to
a06f110
Compare
…ejs-firestore into tomandersen/count
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've reviewed the non-test code so far. I'm not done my review, but wanted to send my feedback now because I probably won't get back to it until later today.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you fix the CI failures? ( |
6a76cdd
to
d12c82a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I've added the "do not merge" label. Since the PR exposes the API publicly we should wait until allowed to merge the PR.
Denver is away, and won't be able to approve.
Add support for "count" queries; that is, a query that gets the number of documents in the result set without actually downloading the documents.