Rel 7 0 mb 2 #5714
Merged
Rel 7 0 mb 2 #5714
Conversation
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
* pom bump to 7.0.0 * add version enum * fixed feb release name --------- Co-authored-by: Long Ma <[email protected]>
* Add conditional check on index existence before we try again * Add conditional check on index existence before we try again * Changelog * remit * remit * debug statements
…onal create (#5622) * Modifying the CacheProvider to avoid doing db I/O within the cache miss value supplier callback. * Setting the initial capacity of instantiated caches to the cache max size to avoid resizing during operations. * adding changelog and spotless. * Fixing typo. * Addressing comments from code review. --------- Co-authored-by: peartree <[email protected]>
…ncorrect results (#5614) * Failing test * fix * changelog
* replaced EventListener annotation with @PreDestroy * adding changelog --------- Co-authored-by: peartree <[email protected]>
Co-authored-by: Long Ma <[email protected]>
…with urn id placeholder (#5625) * Incorrect version from versioned_references.auto_at_paths for conditional update - implementation
…tes (#5629) * First pass at fix to Oracle HFJ_RES_VER.RES_TEXT_VC migration. * First stab at agreed upon solution. * Fix error with 4001 by removing unnecessary annotation. * Spotless and TODO. * Remove annotation for good and set length to LONG32. * Fix copyright year. * Finalize changelog. * Remove migration changes. Fix unit test. * Fix compile error. * Log output. * Refactor resource history code into new ResourceHistoryCalculator. * Spotless. * Convert record to POJO. * Restore pre-17 switch statement. * Finalize new resource history calculator code and tests. * Spotless. * Remove logging. * Update hapi-fhir-docs/src/main/resources/ca/uhn/hapi/fhir/changelog/7_0_0/5633-oracle-hfj-res-ver-clob-migration.yaml Apply code reviewer suggestion Co-authored-by: Michael Buckley <[email protected]> * Code review feedback. --------- Co-authored-by: Michael Buckley <[email protected]>
…n:ietf:bcp:13` (#5638) * When fetching the mimetype code system, return empty CodeSystem with NOTSUPPORTED content. Update expansion logic to handle this case. Add some test cases. * Minor change to fix test * Rename changelog file * Remove TODOs as they're replaced by reported issue * Revert accidental change added with TODO removal
…es (#5637) * $expunge operation ignoring ExpungeThreadCount setting in certain cases - implementation
* Update to new version of clinical reasoning, includes a fix for null pointers on Measure group id * add changelog --------- Co-authored-by: Justin McKelvy <[email protected]>
* Remove parameters clause. * Finalize changelog. Add tests. Finalize implementation. * Undo changes to this test. * Revert all changes to FhirQueryRuleImplTest. Add new RuleImplOpTest. Ensure that proper nested Bundle is created for test and other fixes. * Tweak test. * Use real rule applier in test and remove all mocks.
#5635) * Prevent batch2 job execution to stop for empty chunk when last job step is a reduction. Add output to bulk export result even when empty. * Fix test * Unimportant change to force fresh build * Implement review suggestions --------- Co-authored-by: juan.marchionatto <[email protected]>
* Don't hold locks while adding indices during upgrade. * concurrent indexing is non-transactional in PG.
* First commit with failing test. * More tests and logs. * More logs * Try new solution for BaseTransactionProcessor.performIdSubstitutionsInMatchUrl(). * Simplify solution. Add more tests. * Changelog. * javadoc
…5644) * failing test * another failing test case * fix * changelog * fix bug * spotless * cr
…presence of PatientIdPartitionInterceptor. (#5660)
* Add new senstiive data serializer * Add new senstiive data serializer * Add new senstiive data serializer * Remove dead comments * Change up the test * review comments * wip * Tighten tests and push annotation down * Tighten tests and push annotation down * Changelog * Add test * 7.0.1-SNAPSHOT bump * Error code * Add method used by CDR * add version enum * Fix test
* Bump json-path * Pin parrson * Bump elastic * Bump spring version * Exclude JDBC
* Fix "is-a" ValueSet expansion, add "descendent-of" support * Fixed tests in DSTU3 and R5 * Trigger new build * Revert "Trigger new build" This reverts commit 46c672b. * fix default partition setting on resource (#5617) * fix default partition setting on resource * changelog * Handle DEFAULT partition in rule checker. * Fix spotless --------- Co-authored-by: Michael Buckley <[email protected]> Co-authored-by: James Agnew <[email protected]> * pom bump, doc add, version enum add (#5616) Co-authored-by: Long Ma <[email protected]> * fix default partition setting on resource (#5618) * fix default partition setting on resource * Handle DEFAULT partition in rule checker. Co-authored-by: Ken Stevens <[email protected]> * Add setting to make broker not use JacksonMessageConverter (#5611) * Add setting to make broker not use JacksonMessageConverter * Add changelog * Implement suggestions --------- Co-authored-by: juan.marchionatto <[email protected]> * Fix version * add changelog, add attribution, remove accidental bring-overs * add changelog, add attribution, remove accidental bring-overs * Finish jira section --------- Co-authored-by: Ole Hedegaard <[email protected]> Co-authored-by: Ken Stevens <[email protected]> Co-authored-by: Michael Buckley <[email protected]> Co-authored-by: James Agnew <[email protected]> Co-authored-by: longma1 <[email protected]> Co-authored-by: Long Ma <[email protected]> Co-authored-by: jmarchionatto <[email protected]> Co-authored-by: juan.marchionatto <[email protected]>
…s with NullPointerException (#5681) * Fix NullPointerException when performing a system bulk export in the presence of PatientIdPartitionInterceptor. * Fix NPE on -export-status operation with PatientIdPartitionInterceptor
* update measure bundles and tests for enforced populationId specification on Measure group resources * bump CR version to PRE17 * fix bug in versionEnumTest * Update hapi-fhir-docs/src/main/resources/ca/uhn/hapi/fhir/changelog/7_0_0/5654-measurescorer-bug.yaml Co-authored-by: Tadgh <[email protected]> --------- Co-authored-by: Tadgh <[email protected]>
…deSystem property and convert to them to string type. (#5698) * Revert Remote Terminology lookup operation to support all types of CodeSystem property and convert to them to string type. * Spotless fix * Address code review comments
longma1
requested review from
jamesagnew,
michaelabuckley and
tadgh
as code owners
|
Formatting check succeeded! |
michaelabuckley
approved these changes
Feb 20, 2024
Contributor
michaelabuckley
left a comment
michaelabuckley
left a comment
There was a problem hiding this comment.
No obvious problems.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5714 +/- ##
============================================
+ Coverage 81.32% 83.38% +2.05%
- Complexity 23650 26921 +3271
============================================
Files 1425 1681 +256
Lines 86399 103941 +17542
Branches 11677 13185 +1508
============================================
+ Hits 70265 86671 +16406
- Misses 10947 11618 +671
- Partials 5187 5652 +465 ☔ View full report in Codecov by Sentry. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.