Skip to content

Commit 206ece4

Browse files
Query: Fixes flip of boolean switch for hybridSearchSkipOrderByRewrite (#5139)
## Description Fixes flip of boolean switch for hybridSearchSkipOrderByRewrite. Adds test coverage to ensure that we catch stuff like this in the future. ## Type of change - [x] Bug fix (non-breaking change which fixes an issue) Co-authored-by: Debdatta Kunda <[email protected]>
1 parent 0d22105 commit 206ece4

File tree

3 files changed

+586
-136
lines changed

3 files changed

+586
-136
lines changed

Microsoft.Azure.Cosmos/src/Query/Core/QueryPlan/PartitionedQueryExecutionInfoInternal.cs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,7 @@ public List<Documents.Routing.Range<PartitionKeyInternal>> QueryRanges
2626
set;
2727
}
2828

29-
// Change to the below after Direct package upgrade
30-
// [JsonProperty(Constants.Properties.HybridSearchQueryInfo)]
31-
[JsonProperty("hybridSearchQueryInfo")]
29+
[JsonProperty(Constants.Properties.HybridSearchQueryInfo)]
3230
public HybridSearchQueryInfo HybridSearchQueryInfo
3331
{
3432
get;

Microsoft.Azure.Cosmos/src/Query/Core/QueryPlan/QueryPartitionProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,7 @@ internal TryCatch<PartitionedQueryExecutionInfoInternal> TryGetPartitionedQueryE
246246
bIsContinuationExpected = Convert.ToInt32(isContinuationExpected),
247247
bRequireFormattableOrderByQuery = Convert.ToInt32(requireFormattableOrderByQuery),
248248
bUseSystemPrefix = Convert.ToInt32(useSystemPrefix),
249-
bHybridSearchSkipOrderByRewrite = Convert.ToInt32(!hybridSearchSkipOrderByRewrite),
249+
bHybridSearchSkipOrderByRewrite = Convert.ToInt32(hybridSearchSkipOrderByRewrite),
250250
eGeospatialType = Convert.ToInt32(geospatialType),
251251
ePartitionKind = Convert.ToInt32(partitionKind)
252252
};

0 commit comments

Comments
 (0)