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

docs: edit release note for schema change feature flag #57143

Merged

Conversation

angelapwen
Copy link

@angelapwen angelapwen commented Nov 25, 2020

Release note (sql change): This is an empty commit meant to correct
a mistake in a merged release note in #57040. The original release
note indicates that a database administrator should toggle this
feature flag on and off using
SET CLUSTER SETTING feature.schemachange.enabled, but there
should be a '_' character so that the cluster setting would be:
SET CLUSTER SETTING feature.schema_change.enabled.

Below is the full original release note, with the typo fixed.

Release note (sql change): Adds a feature flag via cluster setting
for all schema change-related features. If a user attempts
to use these features while they are disabled, an error indicating
that the database administrator has disabled the feature is
surfaced.

Example usage for the database administrator:
SET CLUSTER SETTING feature.schema_change.enabled = FALSE;
SET CLUSTER SETTING feature.schema_change.enabled = TRUE;

@angelapwen angelapwen requested review from otan, ericharmeling and a team November 25, 2020 20:09
@angelapwen angelapwen self-assigned this Nov 25, 2020
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@otan
Copy link
Contributor

otan commented Nov 25, 2020

probably put that this is a typo on the release note itself. otherwise the script that aggregates this doesn't see that it's a correction

Release note (sql change): This is an empty commit meant to correct
a mistake in a merged release note in cockroachdb#57040. The original release
note indicates that a database administrator should toggle this
feature flag on and off using
SET CLUSTER SETTING feature.schemachange.enabled, but there
should be a '_' character so that the cluster setting would be:
SET CLUSTER SETTING feature.schema_change.enabled.

Below is the full original release note, with the typo fixed.

Release note (sql change): Adds a feature flag via cluster setting
for all schema change-related features. If a user attempts
to use these features while they are disabled, an error indicating
that the database administrator has disabled the feature is
surfaced.

Example usage for the database administrator:
SET CLUSTER SETTING feature.schema_change.enabled = FALSE;
SET CLUSTER SETTING feature.schema_change.enabled = TRUE;
@angelapwen angelapwen force-pushed the amend-schema-feature-flag-release-note branch from 2e1ea02 to b97c9fb Compare November 25, 2020 23:22
@angelapwen
Copy link
Author

probably put that this is a typo on the release note itself. otherwise the script that aggregates this doesn't see that it's a correction

Good point! Done.

@angelapwen
Copy link
Author

bors r=otan

@craig
Copy link
Contributor

craig bot commented Nov 26, 2020

Build succeeded:

@craig craig bot merged commit 425d8ed into cockroachdb:master Nov 26, 2020
@angelapwen angelapwen deleted the amend-schema-feature-flag-release-note branch February 5, 2021 00:15
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.

3 participants