Estou escrevendo um software que rastreia o uso de medicamentos. Estou usando o JPA para interagir com o banco de dados. Meu modelo consiste em duas entidades: umaPrescription e umDose. CadaPrescription tem uma coleção deDose casos que representam as doses dadas ao paciente como parte desta prescrição da seguinte forma:

Prescription.java

@Entity
@XmlRootElement
public class Prescription {

    private long id;
    private Collection<Dose> doses = new ArrayList<Dose>();
    /**
     * Versioning field used by JPA to track concurrent changes.
     */
    private long version;
    // Other properties omitted...

    @Id
    @GeneratedValue(strategy = GenerationType.TABLE)
    public long getId() {
        return id;
    }

    public void setId(long id) {
        this.id = id;
    }

    // We specify cascade such that when we modify this collection, it will propagate to the DOSE table (e.g. when
    // adding a new dose to this collection, a corresponding record will be created in the DOSE table).
    @OneToMany(mappedBy = "prescription", cascade = CascadeType.ALL)
    public Collection<Dose> getDoses() {
        // todo update to list or collection interface.
        return doses;
    }

    public void setDoses(Collection<Dose> doses) {
        this.doses = doses;
    }

    @Version
    public long getVersion() {
        return version;
    }

    /**
     * Application code should not call this method. However, it must be present for JPA to function.
     * @param version
     */
    public void setVersion(long version) {
        this.version = version;
    }
}

Dose.java

@Entity
@XmlRootElement
public class Dose {

    private long id;
    private Prescription prescription;
    // Other properties omitted...

    @Id
    @GeneratedValue(strategy = GenerationType.TABLE)
    public long getId() {
        return id;
    }

    public void setId(long id) {
        this.id = id;
    }

    @XmlTransient
    @ManyToOne
    @JoinColumn(name = "PRESCRIPTION_ID") // Specifies name of column pointing back to the parent prescription.
    public Prescription getPrescription() {
        return prescription;
    }

    public void setPrescription(Prescription prescription) {
        this.prescription = prescription;
    }

}

A Dose só pode existir no contexto de umPrescriptione, portanto, umDose é inserido indiretamente no banco de dados, adicionando-o à coleção de doses de sua prescrição:

DoseService.java

@Stateless
public class DoseService {

    @PersistenceContext(unitName = "PrescriptionUnit")
    private EntityManager entityMgr;

    /**
     * Insert a new dose for a given prescription ID.
     * @param prescriptionId The prescription ID.
     * @return The inserted {@code Dose} instance if insertion was successful,
     * or {@code null} if insertion failed (if there is currently no doses available for the given prescription ID).
     */
    @TransactionAttribute(value = TransactionAttributeType.REQUIRED)
    public Dose addDose(long prescriptionId) {
        // Find the prescription.
        Prescription p = entityMgr.find(Prescription.class, prescriptionId);
        if (p == null) {
            // Invalid prescription ID.
            throw new IllegalArgumentException("Prescription with id " + prescriptionId + " does not exist.");
        }
        // TODO is this sufficient locking?
        entityMgr.lock(p, LockModeType.OPTIMISTIC_FORCE_INCREMENT);
        Dose d = null;
        if (isDoseAvailable(p)) {
            // A dose is available, create it and insert it into the database.
            d = new Dose();
            // Setup the link between the new dose and its parent prescription.
            d.setPrescription(p);
            p.getDoses().add(d);
        }
        try {
            // Flush changes to database.
            entityMgr.flush();
            return d;
        } catch (OptimisticLockException ole) {
            // Rethrow application-managed exception to ensure that caller will have a chance of detecting failure due to concurrent updates.
            // (OptimisticLockExceptions can be swallowed by the container)
            // See "Recovering from Optimistic Failures" (page 365) in "Pro JPA 2" by M. Keith and M. Schincariol for details.
            throw new ChangeCollisionException();
        }
    }


    /**
     * Checks if a dose is available for a given prescription.
     * @param p The prescription for which to look up if a dose is available.
     * @return {@code true} if a dose is available, {@code false} otherwise.
     */
    @TransactionAttribute(value = TransactionAttributeType.MANDATORY)
    private boolean isDoseAvailable(Prescription p) {
        // Business logic that inspects p.getDoses() and decides if it is safe to give the patient a dose at this time.
    }

}

addDose(long) pode ser chamado simultaneamente. Ao decidir se uma dose está disponível, a lógica comercial inspeciona a coleta de doses da prescrição. A transação falhará se esta coleção for modificada simultaneamente (por exemplo, por chamada simultânea paraaddDose(long)) Eu uso oLockModeType.OPTIMISTIC_FORCE_INCREMENT para conseguir isso (em vez de adquirir um bloqueio de tabela na tabela DOSE). NoPro JPA 2 de Keith e Schincariol Eu li isso:

O bloqueio de gravação garante tudo o que o bloqueio de leitura otimista faz, mas também promete incrementar o campo de versão na transação, independentemente de um usuário ter atualizado a entidade ou não. [...] o caso comum para usar OPTIMISTIC_FORCE_INCREMENT é garantir consistência entre as mudanças no relacionamento da entidade (geralmente são relacionamentos um-para-muitos com chaves estrangeiras de destino) quando no modelo de objeto os ponteiros do relacionamento da entidade mudam, mas no modelo de dados nenhuma coluna na tabela de entidades é alterada.

O meu entendimento deste modo de bloqueio está correto? Minha estratégia atual garante que oaddDose a transação falhará se houver QUALQUER alteração na coleção de doses da prescrição (adicione, remova ou atualize qualquer dose na coleção)?