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

3654 postgres lobs #3656

Merged
merged 9 commits into from
May 30, 2022
Merged

3654 postgres lobs #3656

merged 9 commits into from
May 30, 2022

Conversation

tadgh
Copy link
Collaborator

@tadgh tadgh commented May 30, 2022

  • Modify Reindex to move RES_TEXT to RES_TEXT_VC if needed
  • Add Test
  • Add Changelog

@codecov
Copy link

codecov bot commented May 30, 2022

Codecov Report

Merging #3656 (185ad1c) into rel_5_7 (88df654) will decrease coverage by 0.00%.
The diff coverage is 68.75%.

@@              Coverage Diff              @@
##             rel_5_7    #3656      +/-   ##
=============================================
- Coverage      82.83%   82.82%   -0.01%     
- Complexity     20499    20505       +6     
=============================================
  Files           1371     1371              
  Lines          73648    73663      +15     
  Branches       11071    11074       +3     
=============================================
+ Hits           61005    61015      +10     
- Misses          8390     8393       +3     
- Partials        4253     4255       +2     
Impacted Files Coverage Δ
...main/java/ca/uhn/fhir/jpa/dao/BaseHapiFhirDao.java 88.50% <ø> (ø)
...a/ca/uhn/fhir/jpa/dao/BaseHapiFhirResourceDao.java 92.19% <68.75%> (-0.43%) ⬇️
...n/fhir/jpa/config/HapiFhirHibernateJpaDialect.java 75.00% <0.00%> (-15.63%) ⬇️
.../uhn/fhir/jpa/term/TermDeferredStorageSvcImpl.java 90.14% <0.00%> (+0.93%) ⬆️
...ca/uhn/fhir/jpa/dao/tx/HapiTransactionService.java 79.24% <0.00%> (+5.66%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update edc4c14...185ad1c. Read the comment docs.

@tadgh tadgh merged commit d878ae9 into rel_5_7 May 30, 2022
@tadgh tadgh mentioned this pull request May 31, 2022
tadgh added a commit that referenced this pull request Jul 12, 2022
* fix handling of common search parameters

* Revert "fix handling of common search parameters"

This reverts commit 89c45ee.

* Fix implementation, add test (#3378)

* Fix implementation, add test

* Tighten test

* Rip out dead modules

* Add changelog

* Jr 20220210 handle common search params in contained searches (#3377)

* fix handling of common search parameters

* add support for reference search parameters with multiple paths

* Issue 3357

* Version bump

* Fixed null pointer exception for re-loading subscription on cdr restart and when there's no partition id in the request, and added tests

* added changelogs for this fix

* Fix broken changelog file

* Can't specify specific resource type permissions for bulk export (#3376)

* deny user from exporting without perms

* add unit tests

* add changelog

Co-authored-by: olivia-you <[email protected]>

* Make migration donothing as it was added in error

* Remove ehcache

* Add version.yaml:

* Add to sources/javadocs for dist

* Fix typo

* fix up pom

* wip test removing checkstyle plugin from deployable pom

* remove test pom changes, instead just dont deploy to sonatype

* typo

* Version bump

* Add implementation, testing, and changelog

* Add backport info, make aware of 6_0_0

* add jira ref

* Fix test

* Add versionenum

* Bump dep for spring, add changelog, add versionenum, add changelog folder

* Actually do the version bump

* 3654 postgres lobs (#3656)

* Implementation, test, changelog

* Modify changelog

* Fix changelog

* Add upgrade.md file

* Modify to yaml changes

* Update hapi-fhir-docs/src/main/resources/ca/uhn/hapi/fhir/changelog/5_7_3/3654-lob-problems.yaml

Co-authored-by: Ken Stevens <[email protected]>

* tidy

* more work

Co-authored-by: Ken Stevens <[email protected]>

* Fix bad yaml indent

* Version bump

* Back out the reindex changes introduced in 5.7.3

* update changelog

* version bump

* added fix for subscription in default partition when default partition's id is null

* added changelog

* Add version info and changelog folder

* adding release pipeline yml file fo azure

* Fix parameter count limitation on MSSQL/Oracle large transactions (#3075)

* Fix parameter count limitation on MSSQL/Oracle large transactions

* Changelog fix

* Move changelog

* Force a change to trigger CI

* Test refactoring

* Backport oracle fix

* Fix backport

* upping version to 5.7.5 for release

* add versionenum

* Updating version to: 5.7.6 post release.

* Add query-chunk-aware code

* Add new folder

* Fix up versionenum

* Remove imports

Co-authored-by: Jason Roberts <[email protected]>
Co-authored-by: JasonRoberts-smile <[email protected]>
Co-authored-by: Mark Iantorno <[email protected]>
Co-authored-by: Steven Li <[email protected]>
Co-authored-by: Olivia You <[email protected]>
Co-authored-by: olivia-you <[email protected]>
Co-authored-by: Ken Stevens <[email protected]>
Co-authored-by: Long Ma <[email protected]>
Co-authored-by: James Agnew <[email protected]>
Qingyixia pushed a commit that referenced this pull request Jul 22, 2022
* fix handling of common search parameters

* Revert "fix handling of common search parameters"

This reverts commit 89c45ee.

* Fix implementation, add test (#3378)

* Fix implementation, add test

* Tighten test

* Rip out dead modules

* Add changelog

* Jr 20220210 handle common search params in contained searches (#3377)

* fix handling of common search parameters

* add support for reference search parameters with multiple paths

* Issue 3357

* Version bump

* Fixed null pointer exception for re-loading subscription on cdr restart and when there's no partition id in the request, and added tests

* added changelogs for this fix

* Fix broken changelog file

* Can't specify specific resource type permissions for bulk export (#3376)

* deny user from exporting without perms

* add unit tests

* add changelog

Co-authored-by: olivia-you <[email protected]>

* Make migration donothing as it was added in error

* Remove ehcache

* Add version.yaml:

* Add to sources/javadocs for dist

* Fix typo

* fix up pom

* wip test removing checkstyle plugin from deployable pom

* remove test pom changes, instead just dont deploy to sonatype

* typo

* Version bump

* Add implementation, testing, and changelog

* Add backport info, make aware of 6_0_0

* add jira ref

* Fix test

* Add versionenum

* Bump dep for spring, add changelog, add versionenum, add changelog folder

* Actually do the version bump

* 3654 postgres lobs (#3656)

* Implementation, test, changelog

* Modify changelog

* Fix changelog

* Add upgrade.md file

* Modify to yaml changes

* Update hapi-fhir-docs/src/main/resources/ca/uhn/hapi/fhir/changelog/5_7_3/3654-lob-problems.yaml

Co-authored-by: Ken Stevens <[email protected]>

* tidy

* more work

Co-authored-by: Ken Stevens <[email protected]>

* Fix bad yaml indent

* Version bump

* Back out the reindex changes introduced in 5.7.3

* update changelog

* version bump

* added fix for subscription in default partition when default partition's id is null

* added changelog

* Add version info and changelog folder

* adding release pipeline yml file fo azure

* Fix parameter count limitation on MSSQL/Oracle large transactions (#3075)

* Fix parameter count limitation on MSSQL/Oracle large transactions

* Changelog fix

* Move changelog

* Force a change to trigger CI

* Test refactoring

* Backport oracle fix

* Fix backport

* upping version to 5.7.5 for release

* add versionenum

* Updating version to: 5.7.6 post release.

* Add query-chunk-aware code

* Add new folder

* Fix up versionenum

* Remove imports

Co-authored-by: Jason Roberts <[email protected]>
Co-authored-by: JasonRoberts-smile <[email protected]>
Co-authored-by: Mark Iantorno <[email protected]>
Co-authored-by: Steven Li <[email protected]>
Co-authored-by: Olivia You <[email protected]>
Co-authored-by: olivia-you <[email protected]>
Co-authored-by: Ken Stevens <[email protected]>
Co-authored-by: Long Ma <[email protected]>
Co-authored-by: James Agnew <[email protected]>
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.

2 participants