RDBC-1099 Sync 7.2.3 -> 7.2.6 - #309
Merged
Merged
Conversation
This was referenced Sep 11, 2026
poissoncorp
force-pushed
the
RDBC-1099-sync-7.2.5
branch
2 times, most recently
from
September 16, 2026 13:11
e4b85e1 to
73e01a5
Compare
Everything the C# 7.2.5 patch brought: the CDC Sink client surface, server-wide connection strings and what uses them, SSO on client certificates, reading an agent conversation back and cancelling the tool calls it awaits, the Azure Service Bus queue ETL broker, S3 checksum validation, and the new server constants.
…ensed tests - Export the queue sink and smuggler types from the package root - Gate the licensed queue sink tests behind RAVENDB_LICENSE
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.
- Exercise server-wide connection strings against a licensed server - Export the follow-up types from the package root
…rts nothing 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.
Everything the C# 7.2.6 patch brought: session JsonPatch behind the new SessionPatchBehavior convention with the RFC 6902 operation and batch command under it, a per-turn output schema for an agent conversation, chunk text on embeddings generation, QueryToolFailedException, the vector-search alias fix, and the client version itself.
…tion alone Four constructors took a new parameter in the middle of their existing signature, which breaks anyone passing positionally: output_options on RunConversationOperation and its command, store_chunk_text on EmbeddingsGenerationConfiguration, and disable_checksum_validation on RemoteAttachmentsS3Settings. All four move to the end. Operation.wait_for_completion also started returning the server's result instead of None, which is a change every caller can see. It returns None again; the result now comes from _wait_for_completion_result, and the smuggler is its only caller. Also covers the six symbols the coverage audit found untested: import_incremental, export_to_database, get_conversation_messages, CdcSinkTableLoadState, ConnectionStringUsage.list_from_json and UpdateQueueSinkOperationResult.
- Export the two new types callers reach for by hand - Export the types this branch's own API hands back - Reach the rest of the public surface from the package root - Leave only real gaps in the commented backlog - Guard every export with the import test
CLIENT_VERSION moved to 7.2.6 with the rest of the sync, but setup.py still said 7.2.3.post2, so a release cut straight off this branch would have announced 7.2.6 on the wire while publishing a version PyPI already holds. The two always move together.
poissoncorp
force-pushed
the
RDBC-1099-sync-7.2.5
branch
from
September 16, 2026 13:20
73e01a5 to
db2921d
Compare
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
1 participant
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.
RDBC-1112 Sync Python SDK 7.2.5 -> 7.2.6
RDBC-1099 Sync Python SDK 7.2.3 -> 7.2.5
New features
QueueSinkConfiguration,QueueSinkScript,AddQueueSinkOperation,UpdateQueueSinkOperation,OngoingTaskQueueSink,QueueSinkProcessState,AzureServiceBusSinkSource,DatabaseRecord.queue_sinksstore.smuggler:export,import_data,import_incremental,export_to_database,for_database, typedSmugglerResultAddCdcSinkOperation,UpdateCdcSinkOperation,OngoingTaskCdcSink,GetCdcSinkSchemaOperation,TestCdcSinkMappingOperation,DatabaseRecord.cdc_sinksGet/Put/RemoveServerWideConnectionStringOperation,ServerWideConnectionStringGetConversationMessagesOperation,AiOperations.get_conversation_messagessession.advanced.patch/patch_array/patch_objectbuild RFC 6902 operations instead of a patch script wherever the path and the value allow it, and fall back to the script otherwise; standaloneJsonPatchOperation,JsonPatchDocument,JsonPatchResult,JsonPatchCommandDataAiOutputOptions,run_with_schema,stream_with_schemaAzureServiceBusConnectionSettings+EntraId/Passwordless,QueueBrokerType.AZURE_SERVICE_BUSAPI changes
ConnectionStringused_by, plusConnectionStringUsage/ConnectionStringUsageKindCertificateMetadatausage,sso_identifiers,sso_server_public_key_pinning_hashes,allow_any_sso_server; newSsoProvider,SsoIdentifier,CertificateUsageEditClientCertificateOperation.ParametersCertificateDefinitionOngoingTaskPullReplicationAsSinkhub_cursor,sink_cursorOngoingTaskTypeCDC_SINKS3Settingsdisable_checksum_validation,storage_class,to_remote_attachments_s3_settings()RemoteAttachmentsS3Settingsdisable_checksum_validation,to_s3_settings()AzureSettings/RemoteAttachmentsAzureSettingsDocumentConventionssession_patch_behavior, plusSessionPatchBehavior(JSON_PATCHdefault,JAVA_SCRIPTto opt out)CommandTypeJSON_PATCHravendb: subscriptions, time series, counters, the changes API, index and spatial enums, the exception hierarchy, ETL connection strings for every queue broker, and the server-wide operationsEmbeddingsGenerationConfigurationstore_chunk_textAiConversationcancel_pending_action_toolsDatabaseItemTypeTIME_SERIES_DELETED_RANGES; newDatabaseRecordItemType(39 members)LimitType(59),LicenseLimitException,QueryToolFailedException, all in the dispatcherBreaking changes
Found by AST-comparing every changed signature against
v7.2. All resolved, none ship.output_options(RunConversationOperation+ its command),store_chunk_text(EmbeddingsGenerationConfiguration),disable_checksum_validation(RemoteAttachmentsS3Settings).Operation.wait_for_completion()had started returning the server's result instead ofNone. Reverted; the smuggler reads it through a private_wait_for_completion_result().Deliberate, kept:
JsonPatchcommand by default. Measured against the script on a live server: identical for setting an existing member, creating a new one, overwriting an array element, and writing through a missing intermediate (both error). The one difference is writing past the end of an array, which the script appends and JsonPatch rejects.conventions.session_patch_behavior = SessionPatchBehavior.JAVA_SCRIPTrestores the old path wholesale.from_jsonaccept payloads that previously raised.Minors
CLIENT_VERSION7.2.3 -> 7.2.6, andsetup.pywith it (it still said7.2.3.post2, a version PyPI already holds)__init__.py's commented backlog went from 295 names to 50. Gone: 45 C# interfaces, the C#-language index tasks, Newtonsoft plumbing, namespace placeholders, and names this client already has under another spelling. What is left is real: starting and restoring a backup, deploying an ETL task, database-level analyzers, a few database and server-wide task operationstest_imports.pynow asserts every name the package root binds, so dropping an export fails the suiteadd_from_alias_to_where_tokensdiscardedadd_alias's result so aliases never applied;VectorSearchTokenlost its settings on aliasing;DatabaseRecord.from_jsonrequiredAutoIndexes+LockMode;DatabaseRecord.to_jsoniterated auto-indexes without.items();AutoIndexDefinition.from_jsonsame strictness; backup settings called.to_json()on a null scriptX-Forwarded-For, 3 newSupportedFeaturesv7.2, same failure set in both directions.