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
SubscriptionTopic part3 #4817
SubscriptionTopic part3 #4817
Conversation
fil512
commented
May 4, 2023
•
edited
edited
- Introduce IBaseResource.isDeleted() method and convert code to use it
- Add subscription_topic_troubleshooting log
- No longer rely on ResourceGoneException to detect deleted subscription. Instead check isDeleted().
- Demote unexpected exceptions in HapiTransactionService from error to debug since these exceptions are expected e.g. when checking if a resource has been deleted by catching a ResourceGoneException
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! Approved with a few comments
...ocs/src/main/resources/ca/uhn/hapi/fhir/changelog/6_8_0/4817-deleted-resource-detection.yaml
Show resolved
Hide resolved
hapi-fhir-jpaserver-base/src/main/java/ca/uhn/fhir/jpa/dao/BaseHapiFhirDao.java
Show resolved
Hide resolved
hapi-fhir-storage/src/main/java/ca/uhn/fhir/jpa/subscription/model/CanonicalSubscription.java
Show resolved
Hide resolved
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #4817 +/- ##
============================================
+ Coverage 81.32% 83.13% +1.80%
- Complexity 23650 25293 +1643
============================================
Files 1425 1542 +117
Lines 86399 92274 +5875
Branches 11677 12313 +636
============================================
+ Hits 70265 76710 +6445
+ Misses 10947 10135 -812
- Partials 5187 5429 +242 ☔ View full report in Codecov by Sentry. |
* Force Verify tests * wip * merge troubleshooting rel_6_6 troubleshooting changes * fix ITs (#4809) * fix RestHookTestR5IT * fix intermittent --------- Co-authored-by: Ken Stevens <ken@smilecdr.com> * post-merge cleanup * fix test * fix mock test * fix wiring * fix mock test * fix test * use IBaseResource.isDeleted() * fixmes * cleanup * change log --------- Co-authored-by: Tadgh <garygrantgraham@gmail.com> Co-authored-by: Ken Stevens <ken@smilecdr.com>