Skip to content

Conversation

@boutss
Copy link
Contributor

@boutss boutss commented Oct 21, 2024

He mistakenly assumes, at first glance, that the mappedBy attribut is not a @manytoone or @Any because the mappedBy attribut is mapped in a secondary table and therefore has a join.

The problem appears later during a cast where the instance of type DiscriminatedAssociationAttributeMapping is cast to PluralAttributeMapping.

https://hibernate.atlassian.net/browse/HHH-18750


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


…astException)

He mistakenly assumes, at first glance, that the mappedBy attribut is not a @manytoone or @Any because the mappedBy attribut is mapped in a secondary table and therefore has a join.

The problem appears later during a cast where the instance of type DiscriminatedAssociationAttributeMapping is cast to PluralAttributeMapping.

https://hibernate.atlassian.net/browse/HHH-18750
boutss and others added 2 commits October 22, 2024 10:33
…astException)

He mistakenly assumes, at first glance, that the mappedBy attribut is not a @manytoone or @Any because the mappedBy attribut is mapped in a secondary table and therefore has a join.

The problem appears later during a cast where the instance of type DiscriminatedAssociationAttributeMapping is cast to PluralAttributeMapping.

https://hibernate.atlassian.net/browse/HHH-18750
…astException)

Proposed solution: Follow the same approach as @manytoone for the second pass.

For your information : @onetomany with @manytoone that uses a @SecondaryTable works perfectly.

https://hibernate.atlassian.net/browse/HHH-18750
@boutss
Copy link
Contributor Author

boutss commented Oct 24, 2024

For my information @beikov i don't the right to add Label?
For example "JIRA Issue Test Case".

@beikov beikov merged commit eeaf27e into hibernate:main Dec 27, 2024
19 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants