Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rollback Trx No DataValidation #251

Closed
IS908 opened this issue Jan 23, 2019 · 2 comments
Closed

Rollback Trx No DataValidation #251

IS908 opened this issue Jan 23, 2019 · 2 comments

Comments

@IS908
Copy link

IS908 commented Jan 23, 2019

public abstract class AbstractUndoExecutor {
        public void executeOn(Connection conn) throws SQLException {
                dataValidation(conn);
                xxxxxxxx
        }

        protected void dataValidation(Connection conn) throws SQLException {
                // Validate if data is dirty.
        }
}

对于数据回滚校验,我想通过 重写 struct 下实体类的 equals 方法实现一版,并扩展 force rollback 选项。

@fescar-robot
Copy link

Hi @IS908, we detect non-English characters in the issue. This comment is an auto translation from @fescar-robot to help other users to understand this issue.
We encourage you to describe your issue in English which is more friendly to other users.

Rollback Trx No DataValidation

public abstract class AbstractUndoExecutor {
        public void executeOn(Connection conn) throws SQLException {
                dataValidation(conn);
                xxxxxxxx
        }

        protected void dataValidation(Connection conn) throws SQLException {
                // Validate if data is dirty.
        }
}

For data rollback validation, I want to implement a version by rewriting the equals method of the entity class under struct and extend the force rollback option.

IS908 added a commit to IS908/fescar that referenced this issue Feb 19, 2019
* issue apache#251 add DataValidation # for UnitTest MockConnection force rollback
@leizhiyuan
Copy link
Contributor

fix in #1060

JacksonZhang95 pushed a commit to JacksonZhang95/seata that referenced this issue Nov 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants