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

Lets unit tests validating JpaExtension and unit tests validating PersistenceExtension run side-by-side; continuation of overall fix for nested transaction problems #7118

Merged
merged 2 commits into from
Jul 13, 2023

Conversation

ljnelson
Copy link
Member

@ljnelson ljnelson commented Jun 27, 2023

This PR:

  • deliberately duplicates the chirp series of unit tests as chirp2 with changes appropriate to the more correct transactional semantics that PersistenceExtension enforces
  • ensures that the chirp and chirp2 suites can be run together on all JPA providers
  • exercises and validates the deliberately obscure System property flags that switch JpaExtension and PersistenceExtension (its replacement) on and off, proving they can "live" together

…sistenceExtension run side-by-side; continuation of overall fix for nested transaction problems

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>
@ljnelson ljnelson added jpa/jta 3.x Issues for 3.x version branch labels Jun 27, 2023
@ljnelson ljnelson self-assigned this Jun 27, 2023
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jun 27, 2023
@ljnelson ljnelson added this to Sprint Scope in Backlog Jun 27, 2023
@ljnelson ljnelson marked this pull request as ready for review June 29, 2023 17:27
Copy link
Member

@tjquinno tjquinno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few mostly very minor notes.

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>
@ljnelson ljnelson requested a review from tjquinno July 13, 2023 22:26
@ljnelson ljnelson merged commit 096217b into helidon-io:helidon-3.x Jul 13, 2023
12 checks passed
Backlog automation moved this from Sprint Scope to Closed Jul 13, 2023
ljnelson added a commit to ljnelson/helidon that referenced this pull request Aug 23, 2023
…sistenceExtension run side-by-side; continuation of overall fix for nested transaction problems (helidon-io#7118)

* Lets unit tests validating JpaExtension and unit tests validating PersistenceExtension run side-by-side; continuation of overall fix for nested transaction problems

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>
ljnelson added a commit that referenced this pull request Aug 25, 2023
* Improves integrations/jdbc/jdbc to better support future JPA improvements; initial work (#5654)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Squashable commit; initial work (#5716)

Lays some groundwork with deprecation and cleanup and isolated improvements to support ongoing JPA improvements.

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Introduces LocalXAResource and a few support classes in jta/jdbc. (#5733)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Adds connection unwrapping abilities to CDISEPlatform.java (#5790)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Introduces JtaConnection.java (#5905)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Fixes erroneous closing behavior in JtaConnection.java (#6321)

* Fixes erroneous closing behavior in JtaConnection.java

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Minor JPA cleanups; part of overall refactoring effort (#6435)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Improving JPA pom.xml as part of overall JPA refactoring (#6508)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Fixes merge conflicts etc. from cherry-pick of c9a849e

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Adds an enabled flag to JpaExtension to permit subsequent refactoring and replacement (#6512)

Adds an enabled flag to JpaExtension to permit subsequent refactoring and replacement

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Adds more classes as part of overall JPA refactoring effort (#6584)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Lets unit tests validating JpaExtension and unit tests validating PersistenceExtension run side-by-side; continuation of overall fix for nested transaction problems (#7118)

* Lets unit tests validating JpaExtension and unit tests validating PersistenceExtension run side-by-side; continuation of overall fix for nested transaction problems

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Resolves issue 7316, which features some intermittent database-related tests (#7317)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Addresses copyright plugin complaints after lots of cherry-picking from old 3.x commits

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

---------

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>
dalexandrov pushed a commit to dalexandrov/helidon that referenced this pull request Aug 26, 2023
* Improves integrations/jdbc/jdbc to better support future JPA improvements; initial work (helidon-io#5654)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Squashable commit; initial work (helidon-io#5716)

Lays some groundwork with deprecation and cleanup and isolated improvements to support ongoing JPA improvements.

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Introduces LocalXAResource and a few support classes in jta/jdbc. (helidon-io#5733)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Adds connection unwrapping abilities to CDISEPlatform.java (helidon-io#5790)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Introduces JtaConnection.java (helidon-io#5905)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Fixes erroneous closing behavior in JtaConnection.java (helidon-io#6321)

* Fixes erroneous closing behavior in JtaConnection.java

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Minor JPA cleanups; part of overall refactoring effort (helidon-io#6435)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Improving JPA pom.xml as part of overall JPA refactoring (helidon-io#6508)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Fixes merge conflicts etc. from cherry-pick of c9a849e

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Adds an enabled flag to JpaExtension to permit subsequent refactoring and replacement (helidon-io#6512)

Adds an enabled flag to JpaExtension to permit subsequent refactoring and replacement

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Adds more classes as part of overall JPA refactoring effort (helidon-io#6584)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Lets unit tests validating JpaExtension and unit tests validating PersistenceExtension run side-by-side; continuation of overall fix for nested transaction problems (helidon-io#7118)

* Lets unit tests validating JpaExtension and unit tests validating PersistenceExtension run side-by-side; continuation of overall fix for nested transaction problems

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Resolves issue 7316, which features some intermittent database-related tests (helidon-io#7317)

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

* Addresses copyright plugin complaints after lots of cherry-picking from old 3.x commits

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>

---------

Signed-off-by: Laird Nelson <laird.nelson@oracle.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.x Issues for 3.x version branch jpa/jta OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
Backlog
  
Closed
Development

Successfully merging this pull request may close these issues.

None yet

2 participants