Skip to content

Document vshard.router.map_callrw() #2753

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

Merged
merged 13 commits into from
Apr 12, 2022
Merged

Document vshard.router.map_callrw() #2753

merged 13 commits into from
Apr 12, 2022

Conversation

xuniq
Copy link
Contributor

@xuniq xuniq commented Mar 16, 2022

@xuniq xuniq force-pushed the gh-2015-map-reduce branch from 75bb124 to c3fc636 Compare March 23, 2022 05:13
@xuniq xuniq marked this pull request as draft March 23, 2022 09:00
@xuniq xuniq marked this pull request as ready for review March 23, 2022 15:51
@xuniq xuniq requested a review from veod32 March 23, 2022 15:52
@xuniq xuniq force-pushed the gh-2015-map-reduce branch from 293d139 to e961bdc Compare March 24, 2022 07:25
@xuniq xuniq requested a review from Gerold103 March 29, 2022 13:27

The chosen function is called on the master node of each replicaset with the given arguments.

:param function_name: a function to call on the storages
Copy link
Collaborator

Choose a reason for hiding this comment

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

storages — instances with the vshard role "storage" ? Can you think of another wording for that (here and further)?

Copy link
Contributor

Choose a reason for hiding this comment

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

Storages in this context means "masters of all replicasets". But you normally should think of them as just "storages" or "replicasets". Not about individual nodes.

Copy link
Contributor

@Gerold103 Gerold103 left a comment

Choose a reason for hiding this comment

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

Thanks for the patch!


The chosen function is called on the master node of each replicaset with the given arguments.

:param function_name: a function to call on the storages
Copy link
Contributor

Choose a reason for hiding this comment

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

Storages in this context means "masters of all replicasets". But you normally should think of them as just "storages" or "replicasets". Not about individual nodes.

@github-actions github-actions bot temporarily deployed to branch-gh-2015-map-reduce April 8, 2022 10:32 Inactive
@github-actions github-actions bot temporarily deployed to branch-gh-2015-map-reduce April 8, 2022 12:10 Inactive
@xuniq xuniq force-pushed the gh-2015-map-reduce branch from ca20439 to 1578869 Compare April 12, 2022 06:49
@github-actions github-actions bot temporarily deployed to branch-gh-2015-map-reduce April 12, 2022 06:51 Inactive
@xuniq xuniq requested a review from veod32 April 12, 2022 07:14
@github-actions github-actions bot temporarily deployed to branch-gh-2015-map-reduce April 12, 2022 12:30 Inactive
@xuniq xuniq merged commit 2080a08 into latest Apr 12, 2022
@xuniq xuniq deleted the gh-2015-map-reduce branch April 12, 2022 12:43
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.

[5pt] vshard.router.map_callrw()
3 participants