-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Correction to #6341 #6342
Merged
jamesagnew
merged 1 commit into
7.5.0-snapshot-midcycle-branch
from
ja_20241002_expand_translation_cache_2
Oct 3, 2024
Merged
Correction to #6341 #6342
jamesagnew
merged 1 commit into
7.5.0-snapshot-midcycle-branch
from
ja_20241002_expand_translation_cache_2
Oct 3, 2024
Conversation
This file contains 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
Formatting check succeeded! |
tadgh
approved these changes
Oct 2, 2024
tadgh
approved these changes
Oct 2, 2024
jamesagnew
merged commit Oct 3, 2024
e3ebfce
into
7.5.0-snapshot-midcycle-branch
60 of 63 checks passed
epeartree
pushed a commit
that referenced
this pull request
Oct 16, 2024
* 6323 resource creation deadlock (#6324) * make map reads concurrent * change log * CUstom version number for guaranteed build determinism * licenses * Expand translation cache (#6341) * Expand translation cache * Add changelog * Correction to #6341 (#6342) --------- Co-authored-by: JasonRoberts-smile <[email protected]> Co-authored-by: Tadgh <[email protected]> Co-authored-by: James Agnew <[email protected]>
tadgh
added a commit
that referenced
this pull request
Nov 24, 2024
* 6323 resource creation deadlock (#6324) * make map reads concurrent * change log * CUstom version number for guaranteed build determinism * licenses * Expand translation cache (#6341) * Expand translation cache * Add changelog * Correction to #6341 (#6342) * Contained bug (#6402) * Contained bug * more tests * changelog, tests, implementation * code review * backwards logic * Fix Questionnaire doc (#6400) * fix reindex optimizeStorage=ALL_VERSIONS (#6421) * fixing broken rename of last step of reindex (#6429) * Fixes for the translation of parameter issues as part of the output for $validate-code operation. (#6438) * Move the validation providers to the test utilities package such that they can be reused. * Update IValidationSupport.CodeValidationIssue structure such that it meets the FHIR specification. Update RemoteTerminologyServiceValidationSupport and VersionSpecificWorkerContextWrapper translation for issues. * New tests for issue translation. Move test class to a different package so that we can add another test class. * Some simplification in the issue API to simplify building of issues. * Fix compilation errors. * Update providers to allow multiple responses and add support for the fetchCodeSystem call through method find. * Setup the first test for resource validation with remote terminology providers. * Fix NullPointerException * avoid calling validateCode for CodeSystem where system is null * Keep old public API methods (and class name) in IValidationSupport and mark them as deprecated to avoid breaking dependencies. * Revert local change to debug for duplicate errors. * Add more test cases for resource validation. Throw exception to signal missing test setup to make it obvious. * Simplify test setup. * Add some more javadoc * Add javadoc for the new test class * Add more tests * Address code review comments in IValidationSupport. * Add changelog * Change Repository search interface from Map to Multimap (#6445) Change Map to Multimap to support multiple and clauses. * licenses * fix interceptor hooks from requestDetails not getting called for STORAGE_PRECHECK_FOR_CACHED_SEARCH (#6436) * fix interceptor hooks from requestDetails not getting called for STORAGE_PRECHECK_FOR_CACHED_SEARCH * added unit tests and updated changelog * added one more test case * Add Adapter api (#6450) Lightweight implementation of the adapter pattern. * Bulk Import job status not changed after activation - failing test, fix, changelog (#6452) * Update CR to 3.13.1 (#6433) * Automated Migration Testing (HAPI-FHIR) - updated test migration scripts for 7_4_0 (#6439) * Rel 7 6 CVE (#6446) * Bump commons io * Bump HS and Lucene, and hibernate * Jetty bump for CVE * Resolve CVES * Bump with mismatch lucene * Bump velocity template engine * Revert bom bump * wip * Version bump * move changelog, fix cve * Bump commons-lang * Replace imports * wip * fix HQL break * remove dead code * Fix changelog entry * Bump org.hl7.fhir.core to 6.4.0 (#6454) * Bump HAPI version + org.hl7.fhir.core to 6.4.0 * Apply spotless * Correct a bug with duplicate parser IDs being assigned. (#6456) * Fix changelog entry * wip * compilation problem * Correct tests * changelog * Update hapi-fhir-structures-r4/src/test/java/ca/uhn/fhir/parser/JsonParserR4Test.java Co-authored-by: volodymyr-korzh <[email protected]> * Correct a bug with duplicate parser IDs being assigned - test fixes * Correct a bug with duplicate parser IDs being assigned - spotless * Correct a bug with duplicate parser IDs being assigned - added test-utilities dependency to fhir-structures poms * Correct a bug with duplicate parser IDs being assigned - test fixes * Contained resources without assigned IDs are now assigned GUIDs - address comments --------- Co-authored-by: volodymyr-korzh <[email protected]> Co-authored-by: volodymyr <[email protected]> * licenses * ValueSet expansion fails if Hibernate Search configured to use Lucene - fixed incompatibility between Hibernate Search and Lucene versions (#6468) * Change the migrator to avoid table locks when adding an index. (#6489) * version bump * Updating version to: 7.6.1 post release. * Bump to 7.7.7. * deadsapce * Profile reference param (#6501) * updated QueryStack to not throw error with _profile as a ReferenceParam * update QueryStack * remove warnings * cleanup cast * cleanup test * add changelog * cleanup imports * updates per review feedback * updates per review feedback --------- Co-authored-by: [email protected] <[email protected]> * move changelog --------- Co-authored-by: JasonRoberts-smile <[email protected]> Co-authored-by: James Agnew <[email protected]> Co-authored-by: Brenin Rhodes <[email protected]> Co-authored-by: Emre Dincturk <[email protected]> Co-authored-by: TipzCM <[email protected]> Co-authored-by: Martha Mitran <[email protected]> Co-authored-by: Michael Buckley <[email protected]> Co-authored-by: volodymyr-korzh <[email protected]> Co-authored-by: dotasek <[email protected]> Co-authored-by: volodymyr <[email protected]> Co-authored-by: markiantorno <[email protected]> Co-authored-by: Gary Graham <[email protected]> Co-authored-by: Taha <[email protected]> Co-authored-by: [email protected] <[email protected]>
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.
The fix in #6341 will not work because we short circuit the cache.