user650828 wrote:Better make the DAO a stateless EJB to take advantage of container managed transactions and Inject a Datasource into it using @Resource.
The Dao is itself annotated using @ApplicationScoped and set eager true.
Why not? An EE6 container supports CDI. If you have it then prefer the CDI (unified api) annotations to the JSF ones for those scopes as well.What suggested earlier was using @Inject, but that is totally CDI and i can't use it with current implementation.
The reason i ask this is that with my current approach will this incur performance problem or worse, crash the app server?Well eager will make the bean be created at application server startup but it's used in beans that should not be created at application startup. Doesn't sound write to be initializing that bean there.