JanusGraph/janusgraph

Rollback documentation and code inconsistency

オープン

#1,077 opened on 2018/05/21

 (2 件のコメント) (0 件のリアクション) (0 人の担当者)Java (1,137 件のフォーク)batch import
Hacktoberfestarea/docshelp wanted

Repository metrics

Stars
 (4,985 個のスター)
PR merge metrics
 (平均マージ 3d 18h) (30d で 12 merged PRs)

説明

There is a sentence in the transactions' documentation:

If a commit() transactions fails, it should be terminated manually with rollback() after catching the failure.

And there is a code of commit() implementation:

try {
    if (this.hasModifications()) {
        this.graph.commit(this.addedRelations.getAll(), this.deletedRelations.values(), this);
    } else {
        this.txHandle.commit();
    }

    success = true;
} catch (Exception var9) {
    try {
        this.txHandle.rollback();
    } catch (BackendException var8) {
        throw new JanusGraphException("Could not rollback after a failed commit", var9);
    }

    throw new JanusGraphException("Could not commit transaction due to exception during persistence", var9);
}

As I see here, the rollback() is called if commit() is failed. I think that the documentation should say: If a commit() transactions fails, it will be terminated automatically with rollback() or something like that. Maybe some clarification should be put there.

コントリビューターガイド