Skip to content

feat: Added public preview support for full-text search and geo search in Pipelines.#2346

Merged
MarkDuckworth merged 19 commits intomainfrom
markduckworth/search
Apr 3, 2026
Merged

feat: Added public preview support for full-text search and geo search in Pipelines.#2346
MarkDuckworth merged 19 commits intomainfrom
markduckworth/search

Conversation

@MarkDuckworth
Copy link
Copy Markdown
Contributor

@MarkDuckworth MarkDuckworth commented Mar 23, 2026

search API - features not available at launch have been marked internal or commented out

@MarkDuckworth MarkDuckworth requested a review from a team as a code owner March 23, 2026 23:08
@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 Mar 23, 2026
@MarkDuckworth MarkDuckworth changed the title feat: Add beta support for full-text search and geo search. feat: Added public preview support for full-text search and geo search in Pipelines. Apr 2, 2026
@MarkDuckworth MarkDuckworth requested a review from wu-hui April 2, 2026 19:00
* @param rquery Define the search query using the search domain-specific language (DSL).
* @return A new {@link Expression} representing the snippet operation.
*/
@BetaApi
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I need to make this internal because it will not be supported at launch

@MarkDuckworth MarkDuckworth requested review from cherylEnkidu and removed request for wu-hui April 2, 2026 20:54
public static final QueryEnhancement REQUIRED = new QueryEnhancement("required");

/** Search will use the un-enhanced, user provided query. */
public static final QueryEnhancement DISABLED = new QueryEnhancement("disabled");
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can you let me know what is the reason for not using standard enum like this:

public enum QueryEnhancement {
    PREFERRED("preferred"),
    REQUIRED("required"),
    DISABLED("disabled");
}

I checked the other place like ReplaceWith, and looks like they use standard enum

@MarkDuckworth MarkDuckworth added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 3, 2026
…s required a refactor of BaseTest. Also added logic to test setup to ensure unexpected docs are cleaned up
@MarkDuckworth MarkDuckworth removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 3, 2026
@MarkDuckworth MarkDuckworth merged commit 0c81885 into main Apr 3, 2026
38 of 39 checks passed
@MarkDuckworth MarkDuckworth deleted the markduckworth/search branch April 3, 2026 15:23
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 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.

4 participants