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

Add lastResourceOptimisationInterfaceClassName and commitMarkableResourceJNDINames to JTAEnvironmentBean for better integration with agroal #107

Merged
merged 1 commit into from
Aug 16, 2023

Conversation

graben
Copy link
Contributor

@graben graben commented Aug 2, 2023

Enhancement based on feedback from agroal/agroal#69. Implementation analog to WildFly JTAEnvironmentBeanService.java#L61

@graben graben marked this pull request as draft August 4, 2023 13:17
@graben graben changed the title Set lastResourceOptimisationInterfaceClassName on JTAEnvironmentBean for better integration with agroal WIP Set lastResourceOptimisationInterfaceClassName on JTAEnvironmentBean for better integration with agroal Aug 4, 2023
@graben graben changed the title WIP Set lastResourceOptimisationInterfaceClassName on JTAEnvironmentBean for better integration with agroal WIP Add lastResourceOptimisationInterfaceClassName and commitMarkableResourceJNDINames to JTAEnvironmentBean for better integration with agroal Aug 4, 2023
…urceJNDINames to JTAEnvironmentBean for better integration with agroal
@graben graben marked this pull request as ready for review August 14, 2023 06:45
@graben graben changed the title WIP Add lastResourceOptimisationInterfaceClassName and commitMarkableResourceJNDINames to JTAEnvironmentBean for better integration with agroal Add lastResourceOptimisationInterfaceClassName and commitMarkableResourceJNDINames to JTAEnvironmentBean for better integration with agroal Aug 14, 2023
@graben
Copy link
Contributor Author

graben commented Aug 14, 2023

@Sgitario : Could you plz review and merge

@Sgitario
Copy link
Contributor

@barreiro can you have a final review on this pull request? Thanks!

@graben graben requested a review from barreiro August 14, 2023 12:31
Copy link

@barreiro barreiro left a comment

Choose a reason for hiding this comment

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

LGTM

@graben
Copy link
Contributor Author

graben commented Aug 15, 2023

@Sgitario : Ready to merge and probably do a first release of 3.0?

@Sgitario Sgitario merged commit 290c1a0 into snowdrop:main Aug 16, 2023
1 check passed
@Sgitario
Copy link
Contributor

Merged.
About releasing, cc @jacobdotcosta

@graben graben deleted the lrco branch August 16, 2023 06:13
@graben
Copy link
Contributor Author

graben commented Aug 17, 2023

@Sgitario : Well, last time @jacobdotcosta was asked for a release, nothing happens. Anyone else able to do releases?

@Sgitario
Copy link
Contributor

@Sgitario : Well, last time @jacobdotcosta was asked for a release, nothing happens. Anyone else able to do releases?

No that I'm aware of. @jacobdotcosta is on PTO until 28 Aug.

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.

None yet

3 participants