Skip to content

Use the CRUD module in the 'Creating a sharded cluster' tutorial #4338

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 6 commits into from
Jul 8, 2024

Conversation

andreyaksenov
Copy link
Contributor

@andreyaksenov andreyaksenov commented Jul 3, 2024

Previous update (migrating tutorial to a new config): #4042.

Fixes #4282.

Note that the old sharded_cluster sample still exists and its code snippets are referenced in the Sharding with vshard topic.

@andreyaksenov andreyaksenov force-pushed the sharded_cluster_use_crud branch 2 times, most recently from a48aac8 to a4de561 Compare July 3, 2024 10:40
@andreyaksenov andreyaksenov marked this pull request as ready for review July 3, 2024 11:58
@andreyaksenov andreyaksenov force-pushed the sharded_cluster_use_crud branch from a4de561 to f7b1802 Compare July 3, 2024 12:53
@andreyaksenov andreyaksenov force-pushed the sharded_cluster_use_crud branch from f7b1802 to f42fe02 Compare July 3, 2024 13:13
@andreyaksenov andreyaksenov requested a review from Totktonada July 3, 2024 13:25
Copy link
Member

@Totktonada Totktonada left a comment

Choose a reason for hiding this comment

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

I've glanced over the changes and I've no objections. Thanks!

@andreyaksenov andreyaksenov requested a review from p7nov July 5, 2024 09:14
Copy link
Contributor

@p7nov p7nov left a comment

Choose a reason for hiding this comment

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

LGTM, a couple minor comments


In this tutorial, you get a sharded cluster up and running on your local machine and learn how to manage the cluster using the tt utility.
To enable sharding in the cluster, the :ref:`vshard <vshard>` module is used.
In this tutorial, the following external modules are used:
Copy link
Contributor

Choose a reason for hiding this comment

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

Same introduction in two sentences in a row.

In this tutorial, the following external modules are used:

- :ref:`vshard <vshard>` enables sharding in the cluster.
- `crud <https://github.com/tarantool/crud>`__ allows you to perform CRUD operations in the sharded cluster.
Copy link
Contributor

Choose a reason for hiding this comment

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

"crud allows .. CRUD operations" - the wording seems a bit useless:

  • those who know the CRUD acronym can guess the module purpose by the name
  • those who don't know it aren't getting any useful explanation.
    Maybe write something about data operations or manipulation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed to allows you to manipulate data in a sharded cluster.

@@ -128,6 +131,10 @@ In this section, the following options are configured:
In particular, this option informs other replica set members that the ``replicator`` user should be used to connect to the current instance.
* ``iproto.advertise.sharding`` specifies how to advertise the current instance to a router and rebalancer.

The cluster topology defined in the :ref:`following section <vshard-quick-start-configuring-cluster-topology>` also specifies the ``iproto.advertise.client`` option for each instance.
This option accepts a URI used to advertise the instance to clients.
For example, |tcm_full_name| uses these URIs to :ref:`connect <tcm_connect_clusters>` to cluster instances.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe a link to TCM? Since it was never introduced and newcomers hardly know about it.


sharded_cluster:router-a-001> vshard.router.bootstrap()
sharded_cluster_crud:router-a-001> vshard.router.bootstrap()
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe mention thett rs vshard bootstrap alternative?

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, an idea: I've recently read the vshard.router.bootstrap reference and found it very useful for newcomers. bootstrap is a pretty technical and broad term, while .. and distribute buckets makes is really easy to understand for the first time.

@andreyaksenov andreyaksenov force-pushed the sharded_cluster_use_crud branch from 01c0665 to 2e3a499 Compare July 8, 2024 08:30
@andreyaksenov andreyaksenov force-pushed the sharded_cluster_use_crud branch from 2e3a499 to 98306a5 Compare July 8, 2024 08:42
@andreyaksenov andreyaksenov merged commit 01bc328 into latest Jul 8, 2024
1 check failed
@andreyaksenov andreyaksenov deleted the sharded_cluster_use_crud branch July 8, 2024 08:42
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.

Update the 'Creating a sharded cluster' tutorial to using the CRUD module
3 participants