Skip to content

feat(firestore): Add support for subqueries#7839

Merged
dlarocque merged 27 commits intomainfrom
dl/subq
Apr 8, 2026
Merged

feat(firestore): Add support for subqueries#7839
dlarocque merged 27 commits intomainfrom
dl/subq

Conversation

@dlarocque
Copy link
Copy Markdown
Contributor

@dlarocque dlarocque commented Mar 20, 2026

Adds support for Pipeline subqueries.

Ported from: firebase/firebase-android-sdk#7736

API Proposal: go/firestore-pipelines-subquery

@product-auto-label product-auto-label Bot added the api: firestore Issues related to the Firestore API. label Mar 20, 2026
@dlarocque dlarocque force-pushed the dl/subq branch 2 times, most recently from 850ba24 to dd1a205 Compare March 23, 2026 20:07
Base automatically changed from dl/defer-validation to main March 24, 2026 14:12
@dlarocque dlarocque marked this pull request as ready for review March 26, 2026 20:01
@dlarocque dlarocque requested a review from a team as a code owner March 26, 2026 20:01
Copy link
Copy Markdown
Contributor

@cherylEnkidu cherylEnkidu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread handwritten/firestore/dev/src/pipelines/expression.ts
Comment thread handwritten/firestore/dev/src/pipelines/expression.ts Outdated
@dlarocque dlarocque requested a review from cherylEnkidu April 2, 2026 15:55
Copy link
Copy Markdown
Contributor

@MarkDuckworth MarkDuckworth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with suggestions for ref doc cleanup

Comment thread handwritten/firestore/dev/src/pipelines/expression.ts Outdated
Comment thread handwritten/firestore/dev/src/pipelines/expression.ts Outdated
Comment thread handwritten/firestore/dev/src/pipelines/expression.ts Outdated
Comment thread handwritten/firestore/dev/src/pipelines/pipelines.ts Outdated
Comment thread handwritten/firestore/dev/src/pipelines/pipelines.ts Outdated
Comment thread handwritten/firestore/dev/src/pipelines/pipelines.ts Outdated
Comment thread handwritten/firestore/dev/src/pipelines/pipelines.ts Outdated
Comment thread handwritten/firestore/dev/src/pipelines/pipelines.ts Outdated
Comment thread handwritten/firestore/types/firestore.d.ts
@MarkDuckworth MarkDuckworth added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 2, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 2, 2026
Copy link
Copy Markdown
Contributor

@MarkDuckworth MarkDuckworth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with some minor documentation improvements

Comment thread handwritten/firestore/dev/src/pipelines/pipelines.ts Outdated
Comment thread handwritten/firestore/dev/src/pipelines/pipelines.ts Outdated
Comment thread handwritten/firestore/dev/src/pipelines/pipelines.ts Outdated
Comment thread handwritten/firestore/types/firestore.d.ts Outdated
@dlarocque dlarocque merged commit fb317f8 into main Apr 8, 2026
36 checks passed
@dlarocque dlarocque deleted the dl/subq branch April 8, 2026 20:49
MarkDuckworth added a commit that referenced this pull request Apr 13, 2026
BEGIN_COMMIT_OVERRIDE
feat(firestore): Add support for DML ([09cd24f](09cd24f))
feat(firestore): Add array expressions ([55d6385](55d6385))
feat(firestore): Add support for subqueries ([#7839](#7839)) ([fb317f8](fb317f8))
feat(firestore): Add the parent expression ([f17a791](f17a791))
feat(firestore): Add timestamp expressions ([deeb265](deeb265))
feat(firestore): Added public preview support for full-text search and geo search in Pipelines ([228fe85](228fe85))
fix(firestore): Refactor the parameters for timestamp expressions and fix bug for timeGranularity lowercase
fix(firestore): Update pipeline tests for backend changes
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: firestore Issues related to the Firestore API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants