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

Add docs for ALTER RANGE ... RELOCATE #13842

Merged
merged 1 commit into from
Jun 2, 2022

Conversation

rmloveland
Copy link
Contributor

@rmloveland rmloveland commented May 11, 2022

Fixes DOC-3344

Summary of changes:

  • Update the existing 'ALTER RANGE' page to make it a list of
    subcommands (similar to 'ALTER DATABASE', etc.)

  • Add a page, 'ALTER RANGE ... RELOCATE', with some info and examples
    about this new command.

  • Add some words and back and forth linkage between 'ALTER RANGE
    ... RELOCATE' docs and the existing 'ALTER TABLE
    ... EXPERIMENTAL_RELOCATE' statement docs

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@github-actions
Copy link

github-actions bot commented May 11, 2022

Files changed:

@rmloveland rmloveland requested a review from lunevalex May 11, 2022 20:06
@netlify
Copy link

netlify bot commented May 11, 2022

Netlify Preview

Name Link
🔨 Latest commit c833f13
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-docs/deploys/629787d3db5d8404fabfbd0f
😎 Deploy Preview https://deploy-preview-13842--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 settings.

v22.1/alter-range-relocate.md Outdated Show resolved Hide resolved
v22.1/alter-range-relocate.md Outdated Show resolved Hide resolved
v22.1/alter-range-relocate.md Show resolved Hide resolved
@rmloveland rmloveland force-pushed the 20220511-alter-range-relocate branch from c0acdc7 to c1e3e28 Compare May 18, 2022 14:56
@rmloveland
Copy link
Contributor Author

Hi @lunevalex! thanks for looking this over

does it look good with the latest changes?

v22.1/alter-range-relocate.md Outdated Show resolved Hide resolved
v22.1/experimental-features.md Outdated Show resolved Hide resolved
Fixes DOC-3344

Summary of changes:

- Update the existing 'ALTER RANGE' page to make it a list of
  subcommands (similar to 'ALTER DATABASE', etc.)

- Add a page, 'ALTER RANGE ... RELOCATE', with some info and examples
  about this new command.

- Add some words and back and forth linkage between 'ALTER RANGE
  ... RELOCATE' docs and the existing 'ALTER TABLE
  ... EXPERIMENTAL_RELOCATE' statement docs
@rmloveland rmloveland force-pushed the 20220511-alter-range-relocate branch from c1e3e28 to c833f13 Compare May 20, 2022 15:10
@rmloveland rmloveland requested review from ericharmeling and ianjevans and removed request for ericharmeling May 20, 2022 15:10
@rmloveland
Copy link
Contributor Author

Hi @ianjevans! mind giving this one a look?

@ianjevans
Copy link
Contributor

Ack, how did I miss this one? Sorry, Rich. I had to retrigger a Netlify deploy to view the output.

@rmloveland
Copy link
Contributor Author

Thanks Ian! No worries, I drop stuff like ALL the time

@rmloveland rmloveland merged commit 73669dc into master Jun 2, 2022
@rmloveland rmloveland deleted the 20220511-alter-range-relocate branch June 2, 2022 14:44
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.

4 participants