Skip to content
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

Issue #2195 - Add logicalResourceIds param to $reindex operation #2474

Closed
wants to merge 89 commits into from

Conversation

tbieste
Copy link
Contributor

@tbieste tbieste commented Jun 4, 2021

Issue #2195 - Add logicalResourceIds param to $reindex operation

Signed-off-by: Troy Biesterfeld tbieste@us.ibm.com

prb112 and others added 28 commits May 19, 2021 20:38
#2322

- mutate the exit status during Mapping failure
- add test for $import of missing file

Signed-off-by: Paul Bastide <pbastide@us.ibm.com>
#2400

Signed-off-by: Paul Bastide <pbastide@us.ibm.com>
…/operation/everything/EverythingOperation.java


Signed-off-by: Paul Bastide <pbastide@us.ibm.com>

Co-authored-by: Lee Surprenant <lmsurpre@us.ibm.com>
Patient/$everything without a patientId results in a Null response. #2400
issue 2405 fixed concurrent updates to correctly return 409 on version conflict
issue 2405 fix version concurrency issue for Derby
Signed-off-by: Paul Bastide <pbastide@us.ibm.com>
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
…h/load/ImportJobListener.java


Signed-off-by: Paul Bastide <pbastide@us.ibm.com>

Co-authored-by: Lee Surprenant <lmsurpre@us.ibm.com>
issue 2420 set Derby isolationLevel to avoid deadlock during ingestion
Error on bulk data status call where import did not import anything.
Signed-off-by: John T.E. Timm <johntimm@us.ibm.com>
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
Previously this was flagged as a db2-only test, which helped contribute
to us missing this issue.

Additionally, this test was written to highlight a potential issue but
didn't yet take into account our solution which is to return a 409 for
one of the updates (in the case of a conflict).

The updated test will retry on a tight loop, ensuring that we can
continue to make progress in the face of contentious PUT requests to the
same resource.

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
* lock in to fhir-examples 4.8.2

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>

* Update to limit releases of fhir-examples

Signed-off-by: Paul Bastide <pbastide@us.ibm.com>
Co-authored-by: Paul Bastide <pbastide@us.ibm.com>
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
Issue #2086 - Change default bulkdata directory
…der/impl/S3Provider.java


Signed-off-by: Paul Bastide <pbastide@us.ibm.com>

Co-authored-by: Lee Surprenant <lmsurpre@us.ibm.com>
…der/impl/S3Provider.java


Signed-off-by: Paul Bastide <pbastide@us.ibm.com>
…der/impl/S3Provider.java


Signed-off-by: Paul Bastide <pbastide@us.ibm.com>
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
…der/impl/S3Provider.java


Signed-off-by: Paul Bastide <pbastide@us.ibm.com>
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
@tbieste tbieste self-assigned this Jun 4, 2021
issue 2283 - eliminate redundant reads during ingestion
prb112 and others added 19 commits June 9, 2021 16:16
…ientTestBase.java


Signed-off-by: Paul Bastide <pbastide@us.ibm.com>

Co-authored-by: Troy Biesterfeld <60481063+tbieste@users.noreply.github.com>
FHIRClient.getWebTarget(String baseURL) throws NPE #2450
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
Issue #2445 - Log warning messages about deprecated tables
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
issue #2461 - upgrade to liberty 21.0.0.5
…st (#2464)

* Issue #2462 - updated Institutional EOB constraints and added unit test

Signed-off-by: John T.E. Timm <johntimm@us.ibm.com>

* Issue #2462 - updated CHANGELOG.md

Signed-off-by: John T.E. Timm <johntimm@us.ibm.com>

* Issue #2462 - updated CHANGELOG.md

Signed-off-by: John T.E. Timm <johntimm@us.ibm.com>

* Issue #2462 - more updates

Signed-off-by: John T.E. Timm <johntimm@us.ibm.com>
Signed-off-by: Paul Bastide <pbastide@us.ibm.com>
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
For both the CapabilityStatement and the TerminologyStatement.

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
* #2421 - Add terminology clear-cache operations

The terminology subsystem caches terminology resources by URL and/or URL
+ version. If the user tries to update a CodeSystem or ValueSet resource
keeping the same URL + version, the cache does not get cleared and some
operations (e.g. validate-code) do not return the expected results after
update. This change adds an operation ($clear-cache) that can be called
on either terminology resource and will clear the appropriate caches.

Signed-off-by: Corey Sanders <corey.thecolonel@gmail.com>

* Address test failure due to unexpected test interactions.

Signed-off-by: Corey Sanders <corey.thecolonel@gmail.com>

* Fix issue with operation ordering on clean database

All my testing has been done on a dirty database. There are automated
build failures that appear to be related to test order and a resource
not existing yet on a clean database. Flipping the order of the resource
puts and initial clear cache operations so that the resource is
guaranteed to be in place.

Signed-off-by: Corey Sanders <corey.thecolonel@gmail.com>

* Address code format issues

Signed-off-by: Corey Sanders <corey.thecolonel@gmail.com>

* Split term cache operations into a separate module

Per discussion with the team, the cache operations are being split into
a separate module that will not be included in the default server build.
Since it isn't automatically included, the build automation scripts
needed to be updated.

Signed-off-by: Corey Sanders <corey.thecolonel@gmail.com>

* Update POM version for new module

Signed-off-by: Corey Sanders <corey.thecolonel@gmail.com>

* Fix Windows integration, update README

Signed-off-by: Corey Sanders <corey.thecolonel@gmail.com>

* Apply suggested change

Signed-off-by: Corey Sanders <corey.thecolonel@gmail.com>

* Apply suggested change

Signed-off-by: Corey Sanders <corey.thecolonel@gmail.com>

Co-authored-by: Lee Surprenant <lmsurpre@us.ibm.com>

* change package name from javax.json to jakarta.json

Signed-off-by: Corey Sanders <corey.thecolonel@gmail.com>

Co-authored-by: Lee Surprenant <lmsurpre@us.ibm.com>
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
add file-based bulkdata config to the docker-compose env
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
@tbieste tbieste force-pushed the tbieste-issue-2195 branch from e4df5da to a8372cc Compare June 9, 2021 21:19
@tbieste tbieste changed the base branch from main to robin-perf-eval June 9, 2021 21:20
@tbieste tbieste changed the base branch from robin-perf-eval to main June 9, 2021 21:21
@tbieste tbieste changed the base branch from main to robin-perf-eval June 9, 2021 21:23
@tbieste tbieste force-pushed the tbieste-issue-2195 branch from a8372cc to 4209d5d Compare June 10, 2021 16:51
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
@tbieste tbieste force-pushed the tbieste-issue-2195 branch from 4209d5d to b022ddb Compare June 10, 2021 17:48
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
@tbieste tbieste force-pushed the tbieste-issue-2195 branch from 20b95ca to ae42233 Compare June 11, 2021 16:03
@tbieste tbieste closed this Jun 11, 2021
@tbieste tbieste deleted the tbieste-issue-2195 branch June 15, 2021 14:26
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.

6 participants