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

fix: use executeStatement in SchemaTool #10516

Merged
merged 1 commit into from
Feb 14, 2023
Merged

Conversation

simPod
Copy link
Contributor

@simPod simPod commented Feb 11, 2023

When using master replica connection the query is routed to replica. We need to use executeStatement so mutation queries are routed to master.

@derrabus derrabus added the Bug label Feb 14, 2023
@derrabus derrabus added this to the 2.14.2 milestone Feb 14, 2023
@derrabus derrabus merged commit 5369e4f into doctrine:2.14.x Feb 14, 2023
@derrabus
Copy link
Member

Thank you @simPod!

@simPod simPod deleted the statements branch February 14, 2023 18:23
derrabus added a commit to derrabus/orm that referenced this pull request Feb 28, 2023
* 2.14.x:
  Ignore the cache dir of PHPUnit 10 (doctrine#10546)
  Make data providers static (doctrine#10544)
  Bump dev tools (doctrine#10541)
  Mark SqlWalker methods as not deprecated (doctrine#10540)
  docs: consistency order for docblock in association mapping (doctrine#10534)
  Correct use of PHP attribute
  fix typo in faq.rst (doctrine#10526)
  fix: use executeStatement in SchemaTool (doctrine#10516)
  Write a test in a more specific way
  Put up a warning sign that mapping may not be inherited from transient classes (doctrine#10392)
  Avoid unnecessary information in query hints to improve query cache hit ratio
derrabus added a commit that referenced this pull request Feb 28, 2023
* 2.14.x:
  Ignore the cache dir of PHPUnit 10 (#10546)
  Make data providers static (#10544)
  Bump dev tools (#10541)
  Mark SqlWalker methods as not deprecated (#10540)
  docs: consistency order for docblock in association mapping (#10534)
  Correct use of PHP attribute
  fix typo in faq.rst (#10526)
  fix: use executeStatement in SchemaTool (#10516)
  Write a test in a more specific way
  Put up a warning sign that mapping may not be inherited from transient classes (#10392)
  Avoid unnecessary information in query hints to improve query cache hit ratio
derrabus added a commit to derrabus/orm that referenced this pull request Feb 28, 2023
* 2.15.x:
  Ignore the cache dir of PHPUnit 10 (doctrine#10546)
  Make data providers static (doctrine#10545)
  Make data providers static (doctrine#10544)
  Bump dev tools (doctrine#10541)
  Mark SqlWalker methods as not deprecated (doctrine#10540)
  docs: consistency order for docblock in association mapping (doctrine#10534)
  Correct use of PHP attribute
  fix typo in faq.rst (doctrine#10526)
  fix: use executeStatement in SchemaTool (doctrine#10516)
  Write a test in a more specific way
  Put up a warning sign that mapping may not be inherited from transient classes (doctrine#10392)
  Avoid unnecessary information in query hints to improve query cache hit ratio
@greg0ire greg0ire linked an issue Apr 12, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Permission denied when creating schema for primary/replicas
3 participants