Skip to content

Change lock SQL helper methods to protected#12363

Open
ruudk wants to merge 1 commit into
doctrine:3.6.xfrom
ruudk:patch-1
Open

Change lock SQL helper methods to protected#12363
ruudk wants to merge 1 commit into
doctrine:3.6.xfrom
ruudk:patch-1

Conversation

@ruudk

@ruudk ruudk commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

This allows to extend SingleSelectSqlFinalizer and then overwrite some of the lock SQL's.

In my case, I want to use SKIP LOCKED in MySQL

This allows to extend `SingleSelectSqlFinalizer` and then overwrite some of the lock SQL's.

In my case, I want to use `SKIP LOCKED` in MySQL
@github-actions

Copy link
Copy Markdown
Contributor

There hasn't been any activity on this pull request in the past 90 days, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 7 days.
If you want to continue working on it, please leave a comment.

@github-actions github-actions Bot added the Stale label Apr 30, 2026
@ruudk

ruudk commented Apr 30, 2026

Copy link
Copy Markdown
Contributor Author

I'd like this to be merged

Who can help me out?

Maybe @derrabus?

@derrabus

Copy link
Copy Markdown
Member

An internal trait is clearly not an intended extension point. I really don't think, we should do that.

@derrabus derrabus removed the Stale label Apr 30, 2026
@derrabus derrabus requested a review from greg0ire April 30, 2026 06:45

@greg0ire greg0ire left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What @derrabus said

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.

3 participants