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

deprecate sharding functionality #1508

Merged
merged 1 commit into from
Apr 15, 2022
Merged

Conversation

dmaicher
Copy link
Contributor

Fixes #1467

@dmaicher dmaicher force-pushed the deprecated_shards branch from 060afe7 to 7d339bc Compare April 15, 2022 10:49
@dmaicher dmaicher added this to the 2.7.0 milestone Apr 15, 2022
@dmaicher dmaicher force-pushed the deprecated_shards branch 2 times, most recently from 99963d6 to c8b7fcb Compare April 15, 2022 10:59
@dmaicher dmaicher force-pushed the deprecated_shards branch from c8b7fcb to 522cec3 Compare April 15, 2022 11:00
@dmaicher
Copy link
Contributor Author

What's our plan for dropping support for Doctrine DBAL v2? We could probably do it for 2.8 then? So people still get a chance to see the deprecations triggered here and can upgrade accordingly?

@dmaicher dmaicher marked this pull request as ready for review April 15, 2022 11:03
@ostrolucky
Copy link
Member

If we are going to remove these options, we have to do it in 3.0

@dmaicher
Copy link
Contributor Author

If we are going to remove these options, we have to do it in 3.0

yeah although this cannot work with DBAL 3 I guess it could still be considered a BC break to remove those options. Let's do it on 3.0 then.

@dmaicher dmaicher merged commit 614e95d into doctrine:2.7.x Apr 15, 2022
@dmaicher dmaicher deleted the deprecated_shards branch April 15, 2022 12:45
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.

2 participants