[ http://opensource.atlassian.com/projects/hibernate/browse/HHH-4982?page=c... ] Emmanuel Bernard moved HCANN-10 to HHH-4982: -------------------------------------------- Key: HHH-4982 (was: HCANN-10) Project: Hibernate Core (was: Hibernate Commons Annotations) -- 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.... - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://opensource.atlassian.com/projects/hibernate/browse/HHH-4982?page=c... ] Gail Badner updated HHH-4982: ----------------------------- Affects Version/s: 3.5.5 -- 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.... - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://opensource.atlassian.com/projects/hibernate/browse/HHH-4982?page=c... ] Manson Yeung commented on HHH-4982: ----------------------------------- I also have such problem after upgrading from 3.3.2 to 3.5.6. In 3.3.2, the OneToOne relationship generate the left outer join. In 3.5.6 it becomes inner join. However in the document OneToOne annotation default is optional=true so I think hibernate should not use inner join for such case. Futher testing show that the use of @PrimaryKeyJoinColumn force the sql become inner join. My current workaround is to put the @PrimaryKeyJoinColumn to another side. For querying TestFare: Criteria lvCriteria = this.getSession().createCriteria(TestFare.class); lvCriteria.add(Restrictions.in("id", fareIds)); lvCriteria.addOrder(Order.asc("lastUpdateTime")); return lvCriteria.list(); My original settings (doesn't work in 3.5.6) @Entity @Table(name = "TB_FARE") public class TestFare implements Serializable { @OneToOne(optional=true, fetch = FetchType.EAGER) @PrimaryKeyJoinColumn private TestRemark fareRemarks; ... } @Entity @Table(name = "TB_FARE_REMARK") public class TestRemark { @OneToOne(mappedBy="fareRemarks") private TestFare fare; ... } Generated SQL: select this_.id as id81_1_, this_.is_special_private_fare as is2_81_1_, this_.last_migrated_date as last3_81_1_, this_.last_update_date as last4_81_1_, testremark2_.id as id82_0_, testremark2_.endorsement_remarks as endorsem2_82_0_, testremark2_.flight_modification_remarks as flight3_82_0_, testremark2_.mileage_accrual_remarks as mileage4_82_0_, testremark2_.misc_remarks as misc5_82_0_, testremark2_.refund_remarks as refund6_82_0_, testremark2_.remarks_variables as remarks7_82_0_, testremark2_.ticket_validity_remarks as ticket8_82_0_ from TZ_FLIGHT.TB_FARE this_ inner join TZ_FLIGHT.TB_FARE_REMARK testremark2_ on this_.id=testremark2_.id where this_.id in (?, ?) order by this_.last_update_date asc Changed Settings (work in 3.5.6) @Entity @Table(name = "TB_FARE") public class TestFare implements Serializable { @OneToOne(fetch = FetchType.EAGER, mappedBy="fare") private TestRemark fareRemarks; } @Entity @Table(name = "TB_FARE_REMARK") public class TestRemark { @OneToOne @PrimaryKeyJoinColumn private TestFare fare; ... } Generated SQL: select this_.id as id81_1_, this_.is_special_private_fare as is2_81_1_, this_.last_migrated_date as last3_81_1_, this_.last_update_date as last4_81_1_, testremark2_.id as id82_0_, testremark2_.endorsement_remarks as endorsem2_82_0_, testremark2_.flight_modification_remarks as flight3_82_0_, testremark2_.mileage_accrual_remarks as mileage4_82_0_, testremark2_.misc_remarks as misc5_82_0_, testremark2_.refund_remarks as refund6_82_0_, testremark2_.remarks_variables as remarks7_82_0_, testremark2_.ticket_validity_remarks as ticket8_82_0_ from TZ_FLIGHT.TB_FARE this_ left outer join TZ_FLIGHT.TB_FARE_REMARK testremark2_ on this_.id=testremark2_.id where this_.id in (?, ?) order by this_.last_update_date asc I am not sure if this is a bug or a designed behaviour on @PrimaryKeyJoinColumn It is used to join the primary table of an entity subclass in the JOINED mapping strategy to the primary table of its superclass; it is used within a SecondaryTable annotation to join a secondary table to a primary table; and it may be used in a OneToOne mapping in which the primary key of the referencing entity is used as a foreign key to the referenced entity. So it would be normal to use inner join if use in superclass-subclass relationship but cause troubles when using in OneToOne mapping. So I think it would be better to allow optional=true to override the inner join behaviour of @PrimaryKeyJoinColumn -- 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.... - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://opensource.atlassian.com/projects/hibernate/browse/HHH-4982?page=c... ] Steve Ebersole updated HHH-4982: -------------------------------- Suitable for new contributors: [Yes, likely] Fix Version/s: 4.0.0.next Assignee: Gail Badner Labels: jpa2 (was: ) Please attach a test case illustrating the reported bug. Please see http://www.hibernate.org/issuetracker.html#testcases -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://opensource.atlassian.com/projects/hibernate/browse/HHH-4982?page=c... ] Gail Badner updated HHH-4982: ----------------------------- Suitable for new contributors: (was: [Yes, likely]) Fix Version/s: (was: 4.0.0.CR2) 4.0.0.next This will have to wait until we have a response about how @NotFound functionality relates to the spec with respect to an optional one-to-one on a primary key. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://opensource.atlassian.com/projects/hibernate/browse/HHH-4982?page=c... ] Gail Badner commented on HHH-4982: ---------------------------------- optional=true was broken by the fix for HHH-5109. For now, I'll back out of the fix for HHH-5109. A new fix for HHH-5109 that does not break optional=true will be covered by HHH-6723. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://opensource.atlassian.com/projects/hibernate/browse/HHH-4982?page=c... ] Gail Badner updated HHH-4982: ----------------------------- Fix Version/s: 3.6.8 -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://opensource.atlassian.com/projects/hibernate/browse/HHH-4982?page=c... ] Gail Badner edited comment on HHH-4982 at 10/12/11 6:02 AM: ------------------------------------------------------------ Backed out of HHH-5109 in master and 3.6. was (Author: gbadner): Backed out of HHH-5100 in master and 3.6. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira