Skip to content

Conversation

ottok
Copy link
Contributor

@ottok ottok commented Feb 1, 2025

Fix trivial spelling.

Originally submitted at openark/orchestrator#1506

@ottok ottok force-pushed the fix-spelling-percona branch from 13525d2 to e795eea Compare February 1, 2025 17:53
Matches all replicas of a given instance under another (destination) instance. This is a (faster) shortcut
to matching said replicas one by one under the destination instance. In fact, this bulk operation is highly
optimized and can execute in orders of magnitue faster, depeding on the nu,ber of replicas involved and their
optimized and can execute in orders of magnitue faster, depending on the nu,ber of replicas involved and their
Copy link

Choose a reason for hiding this comment

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

nu,ber > number

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for spotting one more typo, I included fixing that in this commit now.

@kamil-holubicki kamil-holubicki self-requested a review February 7, 2025 09:26
Copy link
Contributor

@kamil-holubicki kamil-holubicki left a comment

Choose a reason for hiding this comment

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

Hi @ottok , thank you for your contribution. Looks fine. Could you please address the comment from @hulyav ? (nu,ber > number). Then I will merge.

@ottok ottok force-pushed the fix-spelling-percona branch from e795eea to 3367287 Compare February 7, 2025 17:04
@ottok
Copy link
Contributor Author

ottok commented Feb 7, 2025

I extended the commit to include that one new fix and rebased on latest master. Thanks!

@kamil-holubicki kamil-holubicki merged commit 3a90479 into percona:master Mar 28, 2025
2 checks passed
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 this pull request may close these issues.

3 participants