Skip to content

Add support for "vshard_router" option#473

Merged
nickkkccc merged 1 commit intomasterfrom
gh-463
Jan 26, 2024
Merged

Add support for "vshard_router" option#473
nickkkccc merged 1 commit intomasterfrom
gh-463

Conversation

@nickkkccc
Copy link
Copy Markdown
Contributor

I haven't forgotten about:

  • Tests
  • Changelog
  • Documentation
  • Commit messages comply with the guideline
  • Cleanup the code for review. See checklist

Related issues:
Closes #463

@nickkkccc nickkkccc requested review from ArtDu and akudiyar January 19, 2024 14:07
@nickkkccc nickkkccc self-assigned this Jan 19, 2024
@nickkkccc nickkkccc linked an issue Jan 19, 2024 that may be closed by this pull request
@akudiyar
Copy link
Copy Markdown
Collaborator

@nickkkccc I see several PRs that look very dependent on each other. Shouldn't we mark the potentially conflicting ones as drafts or even squash some into one?

@nickkkccc
Copy link
Copy Markdown
Contributor Author

nickkkccc commented Jan 19, 2024

@akudiyar fact, the tasks are independent. It will conflict, but only because the changes will be on the same line. This decision was made to make it easier to check.

ArtDu
ArtDu previously approved these changes Jan 22, 2024
@nickkkccc nickkkccc added this pull request to the merge queue Jan 26, 2024
Merged via the queue into master with commit 00dee2b Jan 26, 2024
@nickkkccc nickkkccc deleted the gh-463 branch January 26, 2024 07:03
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.

Support "vshard_router" CRUD operation option

3 participants