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

Revert "Work around https://hibernate.atlassian.net/browse/HHH-16832" #34465

Merged
merged 1 commit into from
Sep 21, 2023

Conversation

yrodiere
Copy link
Member

@yrodiere yrodiere commented Jul 3, 2023

This reverts commit 823591f.

As suggested here after the upgrade to Hibernate ORM 6.2.6.Final.

@yrodiere yrodiere requested a review from Sanne July 3, 2023 06:39
@quarkus-bot quarkus-bot bot added the area/spring Issues relating to the Spring integration label Jul 3, 2023
@quarkus-bot

This comment has been minimized.

@Sanne
Copy link
Member

Sanne commented Jul 3, 2023

@yrodiere CI seems unhappy about it.. could you double-check the effectiveness of the fix?

@yrodiere
Copy link
Member Author

yrodiere commented Jul 3, 2023

Sorry, can't do right now, too many urgent things to do. I'll get back to it when possible.

@yrodiere
Copy link
Member Author

yrodiere commented Aug 4, 2023

So the fix worked only partially, we still need to fix https://hibernate.atlassian.net/browse/HHH-17034

@yrodiere yrodiere added the triage/on-ice Frozen until external concerns are resolved label Aug 4, 2023
@yrodiere yrodiere marked this pull request as draft August 4, 2023 16:27
@Sanne
Copy link
Member

Sanne commented Sep 19, 2023

Let's rebase this after #35690

@yrodiere yrodiere marked this pull request as ready for review September 20, 2023 08:19
@yrodiere
Copy link
Member Author

Let's rebase this after #35690

Done, let's see.

@quarkus-bot

This comment has been minimized.

@yrodiere
Copy link
Member Author

It works now, but I'll need some approval before I can merge :) @Sanne?

@gastaldi gastaldi added triage/waiting-for-ci Ready to merge when CI successfully finishes and removed triage/on-ice Frozen until external concerns are resolved labels Sep 20, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Sep 21, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@gsmet gsmet merged commit 268dda7 into quarkusio:main Sep 21, 2023
19 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.5 - main milestone Sep 21, 2023
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 21, 2023
@yrodiere yrodiere deleted the revert-HHH-16832-workaround branch January 29, 2024 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/spring Issues relating to the Spring integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants