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(firestore): Add support for PartitionQuery #4206
Conversation
…pecific locations for CMEK
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.
Code generally looks fine, would like some more docs to understand better what this is for!
Also, just realized there was #3476 previously (integration test there is interesting). |
… into partition-query
…so paths are relative for start/end val
… need to try and interpret later if a path is valid
93146fd
to
f145f48
Compare
f145f48
to
1d74b53
Compare
…e rest as internal impl?
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.
Looks good, a couple more small comments. I agree that this surface seems better.
// TODO: add a fromProto to compliment toProto for query. With that serialization is | ||
// very possible. user could also do json.Marshal(toProto()) | ||
|
||
// GetPartitionedQueries returns a slice of Query objects, each containing a |
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.
Since all the other types/methods are now non-exported, this is the only thing that will go into the public package docs. Is there more to add here? Is there a link to the docs on cloud site for this feature?
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.
At the current point we don't have any docs really, other than rest docs, which I am not sure will bring clarity: https://firebase.google.com/docs/firestore/reference/rest/v1beta1/projects.databases.documents/partitionQuery
Other implementations of this:
Java
Merged googleapis/java-firestore@3996548
googleapis/java-firestore#202 (“add support for QueryPartition API”)
googleapis/java-firestore#418 (“verify page count before invoking QueryPartition RPC”)
googleapis/java-firestore#417 (“add implicit ordering for startAt(DocumentReference) calls”)
Released in java-firestore v2.0.0
Node.js
googleapis/nodejs-firestore#1320 (“add support for QueryPartition API”)
googleapis/nodejs-firestore#1328 (“add implicit ordering for startAt(DocumentReference) calls”)
Merged. https://github.com/googleapis/nodejs-firestore/pull/1110/files#diff-487a284b3e5f6cec3fe8dc89ea74120d