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] [#11472] Removed manual transaction table creation from geo-partioning example #11474

Merged
merged 4 commits into from
Mar 4, 2022

Conversation

es1024
Copy link
Contributor

@es1024 es1024 commented Feb 14, 2022

Resolves #11472

Removed the manual transaction table creation process from the Row-Level Geo-Partitioning explore page and added a note in the documentation for yb-admin create_transaction_table indicating that it is not necessary for YSQL.

@netlify /latest/explore/multi-region-deployments/row-level-geo-partitioning/

@es1024 es1024 added the area/documentation Documentation needed label Feb 14, 2022
@netlify
Copy link

netlify bot commented Feb 14, 2022

✔️ Deploy Preview for infallible-bardeen-164bc9 ready!

🔨 Explore the source changes: 9ee57d5

🔍 Inspect the deploy log: https://app.netlify.com/sites/infallible-bardeen-164bc9/deploys/6222326ba15779000881aa16

😎 Browse the preview: https://deploy-preview-11474--infallible-bardeen-164bc9.netlify.app/latest/explore/multi-region-deployments/row-level-geo-partitioning/

Copy link
Contributor

@polarweasel polarweasel left a comment

Choose a reason for hiding this comment

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

Small changes and a question or two...

@polarweasel
Copy link
Contributor

Converting to draft for now, so we don't accidentally merge until 2.12 docs are shipped...

@polarweasel polarweasel marked this pull request as draft February 16, 2022 19:50
@es1024 es1024 force-pushed the docs-tablespace-txntables branch from 806ff01 to 3b3f51f Compare March 2, 2022 15:36
es1024 and others added 2 commits March 2, 2022 07:37
@polarweasel polarweasel marked this pull request as ready for review March 4, 2022 16:48
Copy link
Contributor

@polarweasel polarweasel left a comment

Choose a reason for hiding this comment

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

LGTM. Ready to merge?

@polarweasel polarweasel merged commit da7ef8a into yugabyte:master Mar 4, 2022
@es1024 es1024 deleted the docs-tablespace-txntables branch March 5, 2022 02:09
jayant07-yb pushed a commit to jayant07-yb/yugabyte-db that referenced this pull request Mar 8, 2022
…m geo-partioning example (yugabyte#11474)

Co-authored-by: Alex Ball <aball@yugabyte.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Documentation needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Docs] Remove manual transaction table creation from geo-partioning example
2 participants