[zanata-bugs] [Bug 1017089] New Kerberos user failed to be enabled

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1017089



--- Comment #2 from Sean Flanigan <sflaniga at redhat.com> ---
This should be fixed by
https://github.com/zanata/zanata-server/commit/27711d6b4c9019a00e1f78a9447215275895919e
which reverts the Lombok/field-based changes to our entities.

Apparently Hibernate's dirtiness detection isn't triggered when using
field-based reflection from Seam's IdentityManager.enableUser() and grantRole()
(probably because they never call *any* methods on the entities).  Moving the
relevant Seam annotations for HAccount from the fields (HAccount.enabled and
HAccount.roles) back to accessor methods seems to fix it.

However, due to https://hibernate.atlassian.net/browse/HHH-3718 the use of
field access for the id field of entities could be causing a performance
problem by preventing lazy loading.  See also
http://blog.xebia.com/2009/06/13/jpa-implementation-patterns-field-access-vs-property-access/
and http://stackoverflow.com/a/3739197/14379 

I suggest we try field access again for Zanata 3.2, but with annotated methods
and property access just for the Seam identity properties and for all the
entity id fields.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=qCfkLXFWjm&a=cc_unsubscribe