the point was to generate only if null - so generation would not happen if id is retrieved from database. Current impl wil generate UUID and then replace it with value stored in database. See https://issues.jboss.org/browse/AGPUSH-444