Skip to content

RDBC-1099 Sync 7.2.3 -> 7.2.6 - #309

Open
poissoncorp wants to merge 30 commits into
ravendb:v7.2from
poissoncorp:RDBC-1099-sync-7.2.5
Open

RDBC-1099 Sync 7.2.3 -> 7.2.6#309
poissoncorp wants to merge 30 commits into
ravendb:v7.2from
poissoncorp:RDBC-1099-sync-7.2.5

Conversation

@poissoncorp

Copy link
Copy Markdown
Contributor

RDBC-1112 Sync Python SDK 7.2.5 -> 7.2.6
RDBC-1099 Sync Python SDK 7.2.3 -> 7.2.5

from_json required an AutoIndexes key and a LockMode, so any payload without
them raised. Both C# fields have no initializer: a missing AutoIndexes leaves
null, and a missing LockMode lands on Unlock. to_json also iterated the
auto-index dict without .items(), so it never serialized one.
Ports RemoteAttachmentExtensions. Backup S3Settings gains the StorageClass it was
missing, and the S3 and Azure from_json / to_json paths stop requiring keys that
C# treats as optional, since a converted instance carries no backup script.
Operation.wait_for_completion now hands back the result the server carried instead
of dropping it, and the smuggler narrows that to a SmugglerResult with per-item-type
counts. No existing caller used the return value.
This test pinned wait_for_completion returning None, which is the behaviour the
previous commit deliberately changed. It now checks the result the server sent.
AiOutputOptions carries a sample object, an explicit JSON schema, or no schema at
all, and run_with_schema / stream_with_schema apply it to a single turn. Nothing is
sent when no options are given, so the agent's own schema stays in charge.
VectorSearchToken inherited the base add_alias, which builds a plain WhereToken and
so dropped the similarity threshold, candidate count and exactness. It now rebuilds
itself with the field qualified.

The caller also discarded what add_alias returned, so no alias was ever applied.
C# assigns the new token back into the list; this does the same.
@poissoncorp poissoncorp changed the title Rdbc 1099 sync 7.2.5 RDBC-1099 Sync 7.2.3 -> 7.2.6 Sep 10, 2026
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.

1 participant