[Cosmos][VectorSearch] Non Streaming Order By Query#40098
Merged
aayush3011 merged 54 commits intoMay 9, 2024
Merged
Conversation
Increment package versions for core releases
* Added Alpha3 Java Media Streaming Events * updating readme to add the media streaming events to remove model --------- Co-authored-by: Vinothini Dharmaraj <v-vdharmaraj@microsoft.com>
…re#40012) Co-authored-by: James Suplizio <jasupliz@microsoft.com>
* SDK Review updates * Update auto-generated models * Add customization * Fix customization * Update package * Update tests * Linting
* release azure-cosmos-test 1.0.0.beta.7 --------- Co-authored-by: annie-mac <xinlian@microsoft.com>
* Fixed existsById API in ReactiveCosmosTemplate * Added changelog
…ith Event recordings (Azure#40029) Co-authored-by: Min Woo Lee 🧊 <77083090+minwoolee-ms@users.noreply.github.com>
* remove additional matrix * Fix session processing and disposition instrumentation * return matrix config * review suggestions
* [CODEOWNERS] Updates for org changes The focus of these changes is to remove an individual who no longer is responsible for the products which their GH account is associated to.
…zure#39744) * Move from using the docker image to java2docfx for docs validation * Temporarily turn on docs processing for template libraries for testing * Actually install the rex validation tool * Fix the if not Test-Path statement * Update java2docfx version and add a couple of diagnostics output lines * Add missing close paren * Ensure that Sort-Object always returns an array even if there's only one item * add another piece of diagnostics output * trying one more thing * remove some diag, add other * Remove the additional diagnostics, add permanent output message * Invoke java -jar on java2docfx to show the help command to ensure the install is okay * fiddling with the java -jar command * Set the working directory to the java2docfx directory before executing the mvn dependency download * Actually create the directory before trying to set location...oops * Update rex validation to verify MAVEN_HOME is set * Updates for Java PR 39875 which had changes from this PR that were more immediate * Update java2docfx version * remove check for MAVEN_HOME which was only for testing * Update the version of java2docfx to test a fix * Update version of java2docfx to 1.0.4 * revert template's ci.yml changes that were only necessary to test java2docfx
Use ClientLogger in testing output
…a deployment script for certificate creation. (Azure#40037)
aayush3011
requested review from
a team,
harsimar,
srnagar and
vcolin7
as code owners
aayush3011
requested review from
a team,
Aashish93-stack,
AikoBB,
RoyHerrod,
abhinav-ghai,
allchiang-msft,
besh2014,
danielav7,
dargilco,
gfeitosa-msft,
ilyapaliakou-msft,
johngallardo,
kirankumarkolli,
maximrytych-ms,
miguhern,
mikehang-msft,
milismsft,
minnieliu,
paolamvhz,
phermanov-msft,
rhurey,
shwali-msft,
sjiherzig and
whisper6284
as code owners
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This update introduces support for non-streaming OrderBy, essential for vector search capabilities. Previously, the SDK operated under the assumption that documents returned in response to ORDER BY queries were fully ordered across all continuations. However, with the newly implemented non-streaming OrderBy feature in the backend, this assumption is no longer valid. The backend communicates this change through a new field in the Query Plan, _hasNonStreamingOrderBy, which is set to true when applicable.
In scenarios where _hasNonStreamingOrderBy is true, a new pipeline stage for non-streaming order by is established. This stage functions as a blocking pipeline stage, similar to the GROUP BY stage, and is designed to accumulate all backend responses before yielding any results.
A new query feature by the name of
NonStreamingOrderByhas also been added..Net PR for the same: Azure/azure-cosmos-dotnet-v3#4362
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines