[hibernate-issues] [Hibernate-JIRA] Created: (HHH-6381) using @SecondaryTable with optional=true not handled for joined subclasses (@Inheritance(strategy = InheritanceType.JOINED)

[David Mansfield (JIRA)]

using @SecondaryTable with optional=true not handled for joined subclasses (@Inheritance(strategy = InheritanceType.JOINED)
---------------------------------------------------------------------------------------------------------------------------

                 Key: HHH-6381
                 URL: http://opensource.atlassian.com/projects/hibernate/browse/HHH-6381
             Project: Hibernate Core
          Issue Type: Bug
          Components: core
    Affects Versions: 4.0.0.Beta2, 3.6.5
         Environment: linux, jdk 1.6, hibernate latest from master, oracle 11gr2
            Reporter: David Mansfield


When using a @SecondaryTable(name="xx), set to optional using @org.hibernate.annotations.Table(name="xx",optional=true) the documentation states that the secondary table row will only be populated if any one of the properties mapped to the secondary table are not-null.  However, this does not work with joined subclasses using the JoinedSubclassEntityPersister.  This is because, internally, the boolean which controls "outer joinability" (called isNullableTable) always returns false.

A fix has been tested which will be put into a github pull request. 

The fix implements isNullableTable to honor the optional flag.

Unfortunately, there is no way I could find to really test whether the code is working, as it requires examining the SQL generated for various operations.  However, I have manually verified the fix.  Please look at the output of the JoinedSubclassAndSecondaryTable test, which has been modified as well, to verify the DML statements.


Fix available for 3.6.5 upon request. github pull will be for 4.0 (master).


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/hibernate/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira