Skip to content

Display whether a query ran on a primary or replica in the profiler#2237

Open
MatTheCat wants to merge 1 commit into
doctrine:3.2.xfrom
MatTheCat:profiler_primary_replica
Open

Display whether a query ran on a primary or replica in the profiler#2237
MatTheCat wants to merge 1 commit into
doctrine:3.2.xfrom
MatTheCat:profiler_primary_replica

Conversation

@MatTheCat

@MatTheCat MatTheCat commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

@MatTheCat MatTheCat force-pushed the profiler_primary_replica branch from c857c2d to b78cab6 Compare June 7, 2026 10:24
@MatTheCat MatTheCat force-pushed the profiler_primary_replica branch from b78cab6 to 6b50cc8 Compare June 7, 2026 10:26
@ostrolucky ostrolucky added the Status: On Hold Most likely waiting for upstream resolution label Jun 8, 2026
@MatTheCat MatTheCat force-pushed the profiler_primary_replica branch from 6b50cc8 to 443cbec Compare June 11, 2026 15:50
fabpot added a commit to symfony/symfony that referenced this pull request Jun 24, 2026
…y (MatTheCat)

This PR was merged into the 8.2 branch.

Discussion
----------

[DoctrineBridge] Store whether a query ran on a primary

| Q             | A
| ------------- | ---
| Branch?       | 8.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | Symfony part of #62509
| License       | MIT

Not sure how to test this: the current `MiddlewareTest` uses SQLite which doesn’t support primary/replicas.

To go with doctrine/DoctrineBundle#2237

Commits
-------

4b13a58 [DoctrineBridge] Store whether a query ran on a primary
@ostrolucky

Copy link
Copy Markdown
Member

Please fix CI

@ostrolucky ostrolucky added Status: Needs Work and removed Status: On Hold Most likely waiting for upstream resolution labels Jun 26, 2026
@MatTheCat MatTheCat force-pushed the profiler_primary_replica branch from 443cbec to 82c132b Compare June 26, 2026 12:16
@MatTheCat

Copy link
Copy Markdown
Contributor Author

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants