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

Separate DBAL and driver interfaces #3591

Closed
morozov opened this issue Jun 5, 2019 · 2 comments
Closed

Separate DBAL and driver interfaces #3591

morozov opened this issue Jun 5, 2019 · 2 comments

Comments

@morozov
Copy link
Member

morozov commented Jun 5, 2019

Please refer to the conversation in #3582 (comment). More details to be added.

While currently the wrapper and driver connections and statements implement the same connection and statement interfaces, they don't implement them in the same way and completely. At the same time, the interfaces represent a superset of the contract of the wrapper and driver classes.

  1. The wrapper Connection class has two pairs of methods: executeQuery() + executeUpdate() and query() + exec() where the pairs are almost identical to the second. The first pair is mostly used at the DBAL level but the second is needed to implement the driver-level interface.
  2. Driver statements have to support all fetch modes exposed via the abstraction (e.g. mixed or class) and therefore implement them instead of having the abstraction implementing the fetch modes.
@morozov
Copy link
Member Author

morozov commented Oct 18, 2020

Closed via #4159.

@morozov morozov closed this as completed Oct 18, 2020
@morozov morozov modified the milestones: 4.0.0, 3.0.0 Oct 18, 2020
@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant