Skip to content
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

[DOC-8549] Document features that require finalization, backward-incompatible changes for v23.2 #18278

Merged
merged 9 commits into from
Feb 23, 2024

Conversation

mdlinville
Copy link
Contributor

@mdlinville mdlinville commented Feb 5, 2024

[DOC-8549] Document features that require finalization, backward-incompatible changes for v23.2

The latter were mostly in place already, but I found a couple missing ones to add.

Source for features that require finalization: https://github.com/cockroachdb/cockroach/blame/release-23.2/pkg/clusterversion/cockroach_versions.go#L448-L506

HTML previews:

Backward-incompatible changes: releases/v23.2.md#v23-2-0-backward-incompatible-changes
Features that require finalization: src/current/v23.2/upgrade-cockroach-version.md#features-that-require-upgrade-finalization

Copy link

github-actions bot commented Feb 5, 2024

Files changed:

Copy link

netlify bot commented Feb 5, 2024

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit a9e0dd1
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-interactivetutorials-docs/deploys/65d8f44df0b75d0008f78147

Copy link

netlify bot commented Feb 5, 2024

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit a9e0dd1
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-api-docs/deploys/65d8f44de269be000825c35a

Copy link

netlify bot commented Feb 5, 2024

Netlify Preview

Name Link
🔨 Latest commit a9e0dd1
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-docs/deploys/65d8f44e67bdac00072bc0fa
😎 Deploy Preview https://deploy-preview-18278--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@fqazi fqazi left a comment

Choose a reason for hiding this comment

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

The system.regionliveness table changes look good

@miraradeva
Copy link

The line for #108190 looks good to me.

@mdlinville mdlinville changed the title [DOC-8549] Document backward-incompatible changes for v23.2 [DOC-8549] Document features that require finalization, backward-incompatible changes for v23.2 Feb 6, 2024
Copy link

@sumeerbhola sumeerbhola left a comment

Choose a reason for hiding this comment

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

AC changes are fine
:lgtm:

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @arulajmani and @miraradeva)

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@mdlinville
Copy link
Contributor Author

Thanks for the reviews!

Copy link

@arulajmani arulajmani left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @mdlinville, @mikeCRL, and @miraradeva)


src/current/v23.2/upgrade-cockroach-version.md line 142 at r2 (raw file):

When upgrading from {{ previous_version }} to {{ page.version.version }}, certain features and performance improvements will be enabled only after finalizing the upgrade, including but not limited to:

- The coalescing of storage ranges for each table, index, or partition (collectively referred to as "schema objects") into a single range when individual schema objects are smaller than the default configured maximum range size (controlled using zone configs, specifically the `range_max_bytes parameter`). This change improves scalability with respect to the number of schema objects, since the underlying range count is no longer a potential performance bottleneck. After finalizing the upgrade to v23.2, you may observe a round of range merges and snapshot transfers. To disable this optimization, set the `spanconfig.storage_coalesce_adjacent.enabled` [cluster setting](https://www.cockroachlabs.com/docs/{{ page.version.version }}/cluster-settings) to `false`. Finalization is not required before disabling the optimization. See the [v23.1 release notes]({% link releases/v23.1.md %}) for `SHOW RANGES` for more details. [#102961][#102961]

nit: instead of saying "finalization is not required", it might be better worded as "this should be done before finalization"

Copy link
Contributor

@mikeCRL mikeCRL 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 few suggestions.

src/current/_includes/releases/v23.2/v23.2.0.md Outdated Show resolved Hide resolved
src/current/_includes/releases/v23.2/v23.2.0.md Outdated Show resolved Hide resolved
@mdlinville mdlinville merged commit 4b91509 into main Feb 23, 2024
6 checks passed
@mdlinville mdlinville deleted the DOC-8549 branch February 23, 2024 19:46
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.

8 participants