You're updating the data container directly here and I have the feeling that this is done outside of the locking boundaries (set by the locking interceptor)... isn't it? If so, there are potential risks in doing that.