Message Title

OGM-642

Emmanuel Bernard, my first idea would be to add this to GridDialect:

 
                                                    enum DuplicateInsertPreventionStrategy {
    NONE, // no checking whatsoever, applying "upsert" semantics basically
    LOOK_UP, // current behavior, explicit check prior to insert;
    NATIVE, // store raises an exception upon insertion of duplicate PK
    ROLLBACK_UPON_DUPLICATE // the store reports back an upsert was an update and allows to rollback the change
}

DuplicateInsertPreventionStrategy getDuplicateInsertPreventionStrategy();
 
                                                

It would return LOOK_UP by default as implemented in BaseGridDialect. In order to support ROLLBACK_UPON_DUPLICATE, we'd also have to change the return type of insertOrUpdateTuple() to signal whether an insert or update was performed.

Surely the details would have to be refined, but that it generally sound sensible?

Add Comment

This message was sent by Atlassian JIRA