Update org.hl7.fhir.core to 6.10.0 - #8207
Open
dotasek wants to merge 8 commits into
Open
Conversation
* GL-9002: add failing tests for batched subscription delivery Adds the TDD Red tests for SMILE-12408. Subscription delivery drains HFJ_RESOURCE_MODIFIED one row at a time, each in its own REQUIRES_NEW transaction with a synchronous broker publish, capping throughput at ~29 rows/s on a broker whose durable publish costs ~34ms. AsyncSubscriptionMessageSubmissionIT gains two tests, both against the existing real H2 + real channel infrastructure: - drainsRowsInOneBatchAndDeliversEveryMessage asserts the page is removed with a single DELETE, using SQL captured off the real connection. It also asserts the captured statement is not a collapsed JDBC batch, so a delete that still runs per row inside one transaction cannot pass. - leavesEveryRowInTheDatabaseAndNextPassDrainsThem asserts no row is lost when the broker rejects a message mid-pass. Today rows committed before the failure are already gone. IChannelProducer gains a default sendAll(List) that loops the existing send(IMessage), so every implementor keeps compiling and behaviour is unchanged. It is named sendAll rather than overloading send because both send(List) and a varargs send make producer.send(any()) an ambiguous method call at existing call sites. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * Bump version to 8.11.20-SNAPSHOT Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * Add missing license header to IMethodBinding Added by the license plugin during a DIST build; the file was missing the standard header every other source file in the module carries. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * Implementation fixes, tests, api changes, version bump * Undo version bump * Address review comments on PR 8206 - @SInCE 8.12.0 on the new batch APIs and settings accessors - Msg.code(3017-3020) on the new error logs - Note that catching Exception when preparing a batch is deliberate - Remove comments covered by the surrounding javadoc Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Collaborator
|
Formatting check succeeded! |
* Make visible for extension while testing. * changelog
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
4 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.
No description provided.