That really depends on your application. If you are using sequencing only then setting the visibility to protected would be fine. Some applications however have natural Ids or Ids provided by the application in this case the setter method should be public.
--Gordon
-----Original Message-----
From: Kem Elbrader [mailto:kem.elbrader_at_gmail.com]
Sent: Tuesday, July 03, 2007 10:12 PM
To: persistence_at_glassfish.dev.java.net
Subject: Visibility of setter method for id field
Should the setter method for the id field be protected?