-
Notifications
You must be signed in to change notification settings - Fork 934
Allow columns index be shared between base and subclass properties #2101
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
Merged
Merged
Changes from all commits
Commits
Show all changes
5 commits
Select commit
Hold shift + click to select a range
3bb7895
Allow columns index be shared between columns from base and subclass …
bahusoid ed87ec3
Use base name for shared index
bahusoid c4a30cd
Shorten DB object names
bahusoid 659be67
Rename base index name only if required (Add UniqueIndexNameForDataba…
bahusoid 9508348
Remove UniqueIndexNameForDatabase (better to add in separate PR if ag…
bahusoid File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
using System; | ||
|
||
namespace NHibernate.Test.NHSpecificTest.GH2099 | ||
{ | ||
abstract class PersistentObject | ||
{ | ||
public virtual Guid Id { get; set; } | ||
public virtual bool PDO_Deleted { get; set; } | ||
} | ||
|
||
class WorkflowInstance : PersistentObject | ||
{ | ||
public virtual bool IsWaiting { get; set; } | ||
} | ||
|
||
class Level3 : WorkflowInstance | ||
{ | ||
public virtual bool IsActive { get; set; } | ||
} | ||
|
||
class WorkflowInstance2 : PersistentObject | ||
{ | ||
public virtual bool IsActive { get; set; } | ||
public virtual bool IsWaiting { get; set; } | ||
} | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,27 @@ | ||
using System.Linq; | ||
using NUnit.Framework; | ||
|
||
namespace NHibernate.Test.NHSpecificTest.GH2099 | ||
{ | ||
[TestFixture] | ||
public class Fixture : BugTestCase | ||
{ | ||
protected override void OnSetUp() | ||
{ | ||
} | ||
|
||
protected override void OnTearDown() | ||
{ | ||
} | ||
|
||
[Test] | ||
public void BaseClassCanShareIndexNameWithSubclass() | ||
{ | ||
var table = cfg.CreateMappings().IterateTables.FirstOrDefault(t => t.Name == "Level3"); | ||
|
||
var index = table.GetIndex("CF_BASE_IDX"); | ||
Assert.That(index?.ColumnSpan, Is.EqualTo(3)); | ||
Assert.That(index.ColumnIterator.FirstOrDefault(c => c.Name == "PDO_Deleted"), Is.Not.Null); | ||
} | ||
} | ||
} |
22 changes: 22 additions & 0 deletions
22
src/NHibernate.Test/NHSpecificTest/GH2099/Mappings.hbm.xml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
<?xml version="1.0" encoding="utf-8" ?> | ||
<hibernate-mapping xmlns="urn:nhibernate-mapping-2.2" assembly="NHibernate.Test" | ||
namespace="NHibernate.Test.NHSpecificTest.GH2099"> | ||
|
||
<class name="PersistentObject" table="PObject" abstract="true" dynamic-update="true" lazy="true"> | ||
<id name="Id" generator="guid.comb" /> | ||
<property name="PDO_Deleted" column="PDO_Deleted" index="IDX_DELETED, CF_BASE_IDX" /> | ||
</class> | ||
|
||
<union-subclass name="WorkflowInstance" table="WInstance" abstract="false" dynamic-update="true" lazy="true" extends="PersistentObject"> | ||
<property name="IsWaiting" column="IsWaiting" index="CF_BASE_IDX" /> | ||
</union-subclass> | ||
|
||
<union-subclass name="WorkflowInstance2" table="WInstance2" abstract="false" dynamic-update="true" lazy="true" extends="PersistentObject"> | ||
<property name="IsActive" column="IsActive" index="CF_BASE_IDX" /> | ||
<property name="IsWaiting" column="IsWaiting" index="CF_BASE_IDX" /> | ||
</union-subclass> | ||
|
||
<union-subclass name="Level3" table="Level3" abstract="false" dynamic-update="true" lazy="true" extends="WorkflowInstance"> | ||
<property name="IsActive" column="IsActive" index="CF_BASE_IDX" /> | ||
</union-subclass> | ||
</hibernate-mapping> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Possible improvement: avoid changing index name for dialects where same index name is allowed for different tables.
According to my tests dialects that require unique index name in database:
Firebird
Oracle
SQLite
PostgreSQL
Support same index name:
SQL Server
SQL Server CE
MySql
SAP SQL Anywhere