Fernando Nasser [14/Aug/07 06:11 PM]
Is this something that we should consider for EAP AS CP01 (code freeze Aug 24th)?
Emmanuel, so you believe I can apply this change to entymanager 3.2.1.GA safely?
I don't remember the 3.2 code base. This area has been constantly improved with feedback from the community. So blindly applying it is a no go, but applying it safely should be fine. The ejb3 tests should be a decent safety net (not sure how complete they are these days on persistence).
As for inclusion, this is a pretty big bug from a user POV. We should add a test in QA for space in names.
Emmanuel Bernard [14/Aug/07 07:17 PM]
I don't remember the 3.2 code base. This area has been constantly improved with feedback from the community. So blindly applying it is a no go, but applying it safely should be fine. The ejb3 tests should be a decent safety net (not sure how complete they are these days on persistence).
As for inclusion, this is a pretty big bug from a user POV. We should add a test in QA for space in names.
Fernando Nasser [28/Nov/07 10:13 AM]
Emmanuel, could you please adjust the patch (and check it of course) so it applies correctly against:
http://prdownloads.sourceforge.net/hibernate/hibernate-entitymanager-3.2.1.GA.tar.gz
Send it to me by e-mail (or attach to this JIRA) and I will take care of building and uploading it to the thirdpart repo as 3.2.1.GA.patch01-brew
But we'd need it today so we can ship this fix with EAP 4.3 RC1.
Cheers,
Fernando
http://fisheye.jboss.org/changelog/Hibernate/?cs=12912
Backporting should not be too hard