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.

Sign up for GitHub

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

Jump to bottom

Refactor the acquisition of ContextManager in JDBC metrics #31809

Closed
wants to merge 18 commits into from
Closed

Refactor the acquisition of ContextManager in JDBC metrics #31809

wants to merge 18 commits into from

Conversation

jiangML
Copy link
Contributor

@jiangML jiangML commented Jun 21, 2024 •
edited
Loading

Fixes #31808.

Changes proposed in this pull request:

  • Refactor the acquisition of ContextManager in JDBC metrics

Before committing this PR, I'm sure that I have checked the following options:

  • My code follows the code of conduct of this project.
  • I have self-reviewed the commit code.
  • I have (or in comment I request) added corresponding labels for the pull request.
  • I have passed maven check locally : ./mvnw clean install -B -T1C -Dmaven.javadoc.skip -Dmaven.jacoco.skip -e.
  • I have made corresponding changes to the documentation.
  • I have added corresponding unit tests for my changes.

@jiangML jiangML added the feature: agent shardingsphere agent label Jun 21, 2024
@jiangML jiangML added this to the 5.5.1 milestone Jun 21, 2024
@jiangML jiangML changed the title Dev Refactor the acquisition of ContextManager in JDBC metrics Jun 24, 2024
FlyingZC and others added 15 commits June 24, 2024 11:49
@FlyingZC @jiangML
Fix mysql truncate E2E (apache#31804)
2a6dcc9
@FlyingZC @jiangML
Fix sonar issue of assertBeginTransaction method (apache#31806)
30b0bb5
@strongduanmu @jiangML
Fix sonar issue in different sql statement implementation (apache#31807)
6b1c3aa
@terrymanu @jiangML
Remove useless mock(AuthorityRule.class) on test cases (apache#31810)
c243536 
* Refactor OKDriverStateTest

* Remove useless mock(AuthorityRule.class) on test cases
@terrymanu @jiangML
Fix sonar issue on EncryptAssignmentTokenGenerator (apache#31811)
7878a18
@terrymanu @jiangML
Fix sonar issue on ClickHouseTest (apache#31812)
3ec3415
@lcjing @jiangML
Add p6spy driver support to ShardingJDBC infra (apache#31814)
d63a8e5 
* add p6spy driver support to ShardingJDBC infra

* remove unnecessary dependency and testcontainers modify
@terrymanu @jiangML
Add default impl for ShardingSphereRule.getAttributes() (apache#31818)
1b8c58b
@terrymanu @jiangML
Refactor SQLFederationRule (apache#31819)
4ee0c77
@linghengqian @jiangML
Allow not setting username in datasource config to fix integration te…
511deee 
…sts of testcontainers-java (apache#31817)
@terrymanu @jiangML
Refactor usage of AuthorityRule.findUser() (apache#31822)
34b9634
@terrymanu @jiangML
Use orElseThrow to instead of optional.isPresent (apache#31823)
50c39b5
@FlyingZC @jiangML
Add judgment for aggregation distinct token generator (apache#31805)
18e842b
@menghaoranss @jiangML
Refactor ComputeNodeInstanceContext (apache#31816)
447e5e6
@jiangML
Fix test error
db1959c
@jiangML jiangML closed this Jun 24, 2024
@jiangML jiangML deleted the dev branch June 24, 2024 03:50
@jiangML jiangML restored the dev branch June 24, 2024 03:51
@jiangML jiangML deleted the dev branch June 24, 2024 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
feature: agent shardingsphere agent
Projects
None yet
Milestone
5.5.1
Development

Successfully merging this pull request may close these issues.

Refactor the acquisition of ContextManager in JDBC metrics
7 participants
@jiangML @FlyingZC @strongduanmu @terrymanu @lcjing @linghengqian @menghaoranss