Skip to content

Conversation

@dreab8
Copy link
Member

@dreab8 dreab8 commented Nov 4, 2025

Fix #2651

The PR implements the Reactive version of JdbcSelectWithActions and of the new PostAction and PreAction (ReactiveLockTimeoutHandler and ReactiveLockTimeoutHandler).

The PR does not implement the Reactive version of CollectionLockingAction but in order to use it a Reactive version of org.hibernate.query.SelectionQuery#setLockScope(Locking.Scope lockScope); method need to be added.

@hibernate-github-bot hibernate-github-bot bot changed the title Implemented Reactive versions of PostAction, PreAction and JdbcSelectWithActions [wip/4.2] Implemented Reactive versions of PostAction, PreAction and JdbcSelectWithActions Nov 4, 2025
@hibernate-github-bot hibernate-github-bot bot added the wip/4.2 Label for pull requests targeting [wip/4.2] branch. label Nov 4, 2025
public String name;

@ManyToOne
@ManyToOne(fetch = FetchType.LAZY)
Copy link
Member

Choose a reason for hiding this comment

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

I don't think you should change the mapping of an existing test. ALso, does it matter if you use lazy here?

Copy link
Member Author

Choose a reason for hiding this comment

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

With the new Lock implementation in ORM, it affects the number of queries executed.
I'm going to revert this change and modify the testFindUpgradeNoWait assertions accrdingly

@dreab8 dreab8 force-pushed the issue#2651_4.2 branch 3 times, most recently from 4189a5f to 229542e Compare November 6, 2025 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wip/4.2 Label for pull requests targeting [wip/4.2] branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants