Loader impls should still check the keys extracted from the ResultSet against the persistence context to see if that entity exists there first. So it sounds like that is not happening in this particular Loader. I forget the details of what we may have discussed.