[PR #890] [MERGED] [DBAL-563] Add general IDENTITY generator type support for sequence emulating platforms #8854

Closed
opened 2026-01-22 16:02:00 +01:00 by admin · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/890
Author: @deeky666
Created: 12/29/2013
Status: Merged
Merged: 12/31/2013
Merged by: @guilhermeblanco

Base: masterHead: DBAL-563


📝 Commits (1)

  • 337857d add general IDENTITY generator type support for sequence emulating platforms

📊 Changes

3 files changed (+97 additions, -62 deletions)

View changed files

📝 lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php (+11 -9)
tests/Doctrine/Tests/ORM/Functional/PostgreSQLIdentityStrategyTest.php (+0 -53)
tests/Doctrine/Tests/ORM/Functional/SequenceEmulatedIdentityStrategyTest.php (+86 -0)

📄 Description

This is a follow-up PR for https://github.com/doctrine/dbal/pull/428 as discussed in DBAL-563 which adds IDENTITY generator type support for platforms that do not natively support identity columns but can emulate them via sequences.
This replaces the current workaround for PostgreSQL SERIAL columns in favour of a more general approach. With this patch now IDENTITY generator can also be used with Oracle which did not work before. Future platforms can also make use of this approach if necessary.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/doctrine/orm/pull/890 **Author:** [@deeky666](https://github.com/deeky666) **Created:** 12/29/2013 **Status:** ✅ Merged **Merged:** 12/31/2013 **Merged by:** [@guilhermeblanco](https://github.com/guilhermeblanco) **Base:** `master` ← **Head:** `DBAL-563` --- ### 📝 Commits (1) - [`337857d`](https://github.com/doctrine/orm/commit/337857dc8a77b443d9f93f8f74f728eceb96421b) add general IDENTITY generator type support for sequence emulating platforms ### 📊 Changes **3 files changed** (+97 additions, -62 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php` (+11 -9) ➖ `tests/Doctrine/Tests/ORM/Functional/PostgreSQLIdentityStrategyTest.php` (+0 -53) ➕ `tests/Doctrine/Tests/ORM/Functional/SequenceEmulatedIdentityStrategyTest.php` (+86 -0) </details> ### 📄 Description This is a follow-up PR for https://github.com/doctrine/dbal/pull/428 as discussed in [DBAL-563](http://www.doctrine-project.org/jira/browse/DBAL-563) which adds IDENTITY generator type support for platforms that do not natively support identity columns but can emulate them via sequences. This replaces the current workaround for PostgreSQL `SERIAL` columns in favour of a more general approach. With this patch now IDENTITY generator can also be used with Oracle which did not work before. Future platforms can also make use of this approach if necessary. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
admin added the pull-request label 2026-01-22 16:02:00 +01:00
admin closed this issue 2026-01-22 16:02:01 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#8854