-
Type:
Bug
-
Resolution: Fixed
-
Priority:
Major - P3
-
Affects Version/s: None
-
Component/s: None
-
Query Integration
-
Fully Compatible
-
ALL
-
v8.2
-
0
-
None
-
3
-
TBD
-
None
-
None
-
None
-
None
-
None
-
None
-
None
Non-assignee request: What is the reason for requesting this backport?
- We knowingly introduced a breaking change in
SERVER-93521which lands in 8.2. This change throws an error for malformed queries that reference "searchScore" or "vectorSearchScore" when unavailable in the pipeline. This should help users identify unintended behavior in their query. However, we decided users should have some more notice in case this may break their queries. This backport will have 8.2 log a warning instead of throwing an error. Then in 8.3, the error will be thrown. In the 8.2 time window, users can fix their queries so they don't fail during the 8.3 upgrade.
Could this introduce any (intentional or unintentional) breaking changes? Please explain.
- No. This removes a breaking change in 8.2.
Could this change unintentionally cause tests to fail or introduce risk to internal tools and test infrastructure?
- We do need to be careful about testing since the error-versus-logging behavior is tricky to test, but I do believe we've been comprehensive enough with the testing.
What is the risk and potential customer impact of backporting this (or not backporting this)?
- If we do not backport this, working queries (that incorrectly reference "searchScore" or "vectorSearchScore" may start breaking during 8.2 upgrade.
- is related to
-
SERVER-93521 Enforce that "searchScore" and "vectorSearchScore" cannot be used when undefined.
-
- Closed
-
-
SERVER-108468 [v8.2] Add warning log for using "searchScore" or "vectorSearchScore" when undefined
-
- Closed
-