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

TAC模式下,实现读已提交的隔离级别,并在 hmily-demo-tac-dubbo 模块增加对应测试案例 #356

Merged
merged 3 commits into from
Jul 24, 2023

Conversation

zkyoma
Copy link
Contributor

@zkyoma zkyoma commented Jul 18, 2023

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

for issue #355

…新增 HmilySelectSQLComputeEngine 类,用来执行 SELECT SQL, 查询对应记录的主键,用于获取全局锁。
@cherrylzhao cherrylzhao self-requested a review July 20, 2023 02:29
/**
* Generate snapshot images.
*
* @param sql the sql
* @param sql the sql
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

多余的空格

/**
* Execute query.
*
* @param connection connection
* @param sql sql
* @param sql sql
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

多余的空格

@cherrylzhao
Copy link
Contributor

代码质量很高,checkstyle简单改改,就可以合了

@codecov-commenter
Copy link

Codecov Report

Merging #356 (7f0ebe1) into master (84cda6f) will not change coverage.
The diff coverage is 0.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##           master    #356    +/-   ##
=======================================
  Coverage    0.00%   0.00%            
=======================================
  Files         435     442     +7     
  Lines        8455    8621   +166     
  Branches     1382    1417    +35     
=======================================
- Misses       8455    8621   +166     
Impacted Files Coverage Δ
...ository/spi/entity/tuple/HmilySQLManipulation.java 0.00% <0.00%> (ø)
...hmily/tac/p6spy/executor/HmilyExecuteTemplate.java 0.00% <0.00%> (ø)
...y/tac/sqlcompute/HmilySQLComputeEngineFactory.java 0.00% <0.00%> (ø)
...mily/tac/sqlcompute/impl/HmilySQLComputeUtils.java 0.00% <0.00%> (ø)
...c/sqlcompute/impl/HmilySelectSQLComputeEngine.java 0.00% <0.00%> (ø)
...del/common/statement/dml/HmilySelectStatement.java 0.00% <0.00%> (ø)
...l/dialect/mysql/dml/HmilyMySQLSelectStatement.java 0.00% <0.00%> (ø)
...dialect/oracle/dml/HmilyOracleSelectStatement.java 0.00% <0.00%> (ø)
...postgresql/dml/HmilyPostgresqlSelectStatement.java 0.00% <0.00%> (ø)
...t/sqlserver/dml/HmilySQLServerSelectStatement.java 0.00% <0.00%> (ø)
... and 8 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@cherrylzhao cherrylzhao merged commit d1bee16 into dromara:master Jul 24, 2023
1 check passed
@zkyoma zkyoma deleted the 355 branch July 25, 2023 12:38
cherrylzhao pushed a commit that referenced this pull request Oct 24, 2023
* 在 HmilyExecuteTemplate 中, 增加对 SELECT SQL 的处理, 返回对应的 SelectStatement; 新增 HmilySelectSQLComputeEngine 类,用来执行 SELECT SQL, 查询对应记录的主键,用于获取全局锁。

* 在 hmily-demo-tac-dubbo 模块增加读已提交隔离级别的的测试案例

* checkstyle后, 代码格式简单调整
cherrylzhao pushed a commit that referenced this pull request Oct 24, 2023
* 在 HmilyExecuteTemplate 中, 增加对 SELECT SQL 的处理, 返回对应的 SelectStatement; 新增 HmilySelectSQLComputeEngine 类,用来执行 SELECT SQL, 查询对应记录的主键,用于获取全局锁。

* 在 hmily-demo-tac-dubbo 模块增加读已提交隔离级别的的测试案例

* checkstyle后, 代码格式简单调整
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants