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

*: Optimize the logic of creating sqlrunner. #229 #230

Merged
merged 1 commit into from
Sep 24, 2021

Conversation

runkecheng
Copy link
Collaborator

What type of PR is this?

Optimize the logic of creating sqlrunner.

Which issue(s) this PR fixes?

Fixes #229

What this PR does?

Summary: Apply the necessary configuration to SQLRunnerFactory to create the corresponding sqlrunner, which simplifies the creation process of sqlrunner.

Special notes for your reviewer?

@runkecheng runkecheng added the enhancement New feature or request label Sep 22, 2021
@runkecheng runkecheng added this to the Next milestone Sep 22, 2021
utils/constants.go Outdated Show resolved Hide resolved
internal/sql_runner.go Outdated Show resolved Hide resolved
@runkecheng runkecheng force-pushed the sqlrunner branch 3 times, most recently from 9a6901f to db0c59f Compare September 23, 2021 02:16
@andyli029 andyli029 modified the milestones: Next, v2.1.0 Sep 24, 2021
internal/query.go Outdated Show resolved Hide resolved
@runkecheng runkecheng force-pushed the sqlrunner branch 3 times, most recently from b278546 to 6c20582 Compare September 24, 2021 03:50
cluster/cluster.go Outdated Show resolved Hide resolved
cluster/syncer/statefulset.go Outdated Show resolved Hide resolved
internal/sql_runner.go Show resolved Hide resolved
cluster/cluster.go Outdated Show resolved Hide resolved
internal/sql_runner.go Outdated Show resolved Hide resolved
internal/sql_runner.go Outdated Show resolved Hide resolved
@acekingke
Copy link
Contributor

LGTM

Copy link
Contributor

@andyli029 andyli029 left a comment

Choose a reason for hiding this comment

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

LGTM.

@andyli029 andyli029 merged commit 09638f8 into radondb:main Sep 24, 2021
@runkecheng runkecheng deleted the sqlrunner branch October 25, 2021 08:41
zhl003 pushed a commit to zhl003/radondb-mysql-kubernetes that referenced this pull request Aug 17, 2022
*: Optimize the logic of creating sqlrunner. radondb#229
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feature] Optimize the logic of creating sqlrunner.
3 participants