v2: removed an experimental typo

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-16193.d33vp1qmzwl1gq.amplifyapp.com

Files changed:

• docs/cql/ddl.rst https://pr-16193.d33vp1qmzwl1gq.amplifyapp.com/cql/ddl

An example would be helpful

CREATE KEYSPACE excalibur
    WITH replication = {'class': 'NetworkTopologyStrategy', 'replication_factor' : 3, 'initial_tablets': 2048}

v3: added an example

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-16193.d33vp1qmzwl1gq.amplifyapp.com

Files changed:

• docs/cql/ddl.rst https://pr-16193.d33vp1qmzwl1gq.amplifyapp.com/cql/ddl

I've left three minor suggestions.
A bigger thing is that this update is going to be part of the docs for Enterprise 2024.1. You can exclude it by placing the content within the .. only:: opensource directive. It will be easy with the paragraph because all you do is:

.. only:: opensource

  The ``initial_tablets`` option :label-caution:`Experimental`
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  The ``initial_tablets`` option is used to specify how many tablets the table is split into.
  It is only valid when ``experimental_features: tablets`` is specified in ``scylla.yaml`` (which
  in turn requires ``consistent_cluster_management: true``. It must be a power of two.
  . . . 

It would be harder with the tables, though.

But if you accept the experimental feature mentioned in the Enterprise docs, we can leave it as is.

I've left three minor suggestions. A bigger thing is that this update is going to be part of the docs for Enterprise 2024.1. You can exclude it by placing the content within the .. only:: opensource directive. It will be easy with the paragraph because all you do is:

I don't understand the reference to 2024.1. Why would it go there? Why is it only opensource? Why would we tag it here?

Docs Preview 📖

Docs Preview for this pull request is available here

Changed Files:

• cql/ddl

Note: This preview will be available for 30 days and will be automatically deleted after that period. You can manually trigger a new build by committing changes.

v4:

• minor edits by @annastuchlik
• noted that the calculation applies to each table independently

Conflicts, please rebase

I don't understand the reference to 2024.1. Why would it go there?

I guess I was wrong and it will be part of 2024.2. Whatever the version, it will be visible in the Enterprise docs when it's merged with enterprise.

Why is it only opensource? Why would we tag it here?

The feature is not opensource as such, but it's experimental (for now). We exclude experimental features from the Enterprise docs, as we claim that Enterprise does not support experimental features. That's the only reason.

Having said that, I think we can merge this PR as is to move forward.

please rebase

Why is it only opensource? Why would we tag it here?

The feature is not opensource as such, but it's experimental (for now). We exclude experimental features from the Enterprise docs, as we claim that Enterprise does not support experimental features. That's the only reason.

We can't edit each piece of documentation separately like that. If you want to exclude experimental features from enterprise, introduce markup to denote experimental features, and post-process the documentation to exclude them.

v5: rebased

We can't edit each piece of documentation separately like that. If you want to exclude experimental features from enterprise, introduce markup to denote experimental features, and post-process the documentation to exclude them.

Yes, I see your point. I've already added that to our backlog.

@tgrabiec ping