borinquenkid commented on code in PR #15568: URL: https://github.com/apache/grails-core/pull/15568#discussion_r3462905979
########## grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/CompositeIdWithManyToOneAndSequenceSpec.groovy: ########## Review Comment: You are right, and this one was a genuine mistake on our side. The PR had inverted the relationship from many-to-one to one-to-many and renamed the test, while the class kept its `...ManyToOne...` name and — tellingly — the H7 twin still had the original many-to-one form. So the change altered what the test covers and actually broke H5/H7 symmetry rather than serving it. Reverted to the 8.0.x form (commit `4ad6c766b6`); it restores the original many-to-one semantics, re-aligns H5 with the H7 twin, and passes (1/1). Thanks for catching it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
