Skip to content
This repository was archived by the owner on May 8, 2026. It is now read-only.

feat: Add pipeline snippets - #2252

Merged
wu-hui merged 3 commits into
googleapis:feat/pipeline/private-previewfrom
morganchen12:mc/pipeline-snippets
Dec 11, 2025
Merged

feat: Add pipeline snippets#2252
wu-hui merged 3 commits into
googleapis:feat/pipeline/private-previewfrom
morganchen12:mc/pipeline-snippets

Conversation

@morganchen12

Copy link
Copy Markdown

No description provided.

@morganchen12
morganchen12 requested review from a team November 6, 2025 22:27
@product-auto-label product-auto-label Bot added size: xl Pull request size is extra large. api: firestore Issues related to the googleapis/java-firestore API. labels Nov 6, 2025
void unnestPreserveEmptyArrayExample() throws ExecutionException, InterruptedException {
// [START unnest_preserve_empty_array]
// Seems like the unnest method is missing:
// https://github.com/googleapis/java-firestore/blob/742fab6583c9a6f9c47cf0496124c3c9b05fe0ee/google-cloud-firestore/src/main/java/com/google/cloud/firestore/Pipeline.java#L995

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@wu-hui FYI, there's an unnest method that takes an Expression and no options but there isn't a method that takes an Expression and also UnnestOptions.

@morganchen12
morganchen12 requested a review from wu-hui November 11, 2025 04:19
@morganchen12 morganchen12 changed the title Add pipeline snippets feat: Add pipeline snippets Dec 10, 2025
@morganchen12

Copy link
Copy Markdown
Author

@wu-hui can you merge this PR?I don't have write access.

@wu-hui
wu-hui merged commit 4d41c82 into googleapis:feat/pipeline/private-preview Dec 11, 2025
11 of 16 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: firestore Issues related to the googleapis/java-firestore API. size: xl Pull request size is extra large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants