Skip to content

Conversation

@AndriySvyryd
Copy link
Member

No description provided.

AndriySvyryd and others added 2 commits May 9, 2025 01:25
…e multiple keywords rather than keyword array (#36000)

Port of #35984
Fixes #35983

Description
Reacting to Cosmos changing the signature of FullTextScore function.

Customer impact
TBD

How found
Partner team ask.

Regression
No

Testing
Extensively tested on EF 10, manual testing on EF9. End-to-end testing is not possible because we can't create containers programmatically (no support for it inside EF Core itself, and the Cosmos SDK which supports it is currently only available in beta, so we can't take dependency on it). Instead, we created containers and data using EF 10, ported all the query tests from EF 10 and ran them using the EF9 bits. Partner team will conduct additional testing on their end.

Risk
Low. Localized change to translation - method signature exposed by EF is already in place. New translation now resembles other FullText methods, so same techniques can be used for translation (essentially re-using code that has been in place for other methods and working). Full text search support is experimental on EF9. Added quirk just in case.
@AndriySvyryd AndriySvyryd requested a review from a team May 15, 2025 22:36
@AndriySvyryd AndriySvyryd enabled auto-merge May 15, 2025 22:48
@AndriySvyryd AndriySvyryd merged commit cd96ce3 into release/9.0 May 15, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants