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

Fix Query Serving Docs #1293

Merged
merged 8 commits into from
Jan 5, 2023
Merged

Conversation

GuptaManan100
Copy link
Member

Description

This PR checks the validity of the current query serving docs

Related Issues

@@ -170,7 +170,7 @@ Vindex Type | Cost

In the case of a simple select, Vitess scans the WHERE clause to match references to Vindex columns and chooses the best one to use. If there is no match and the query is simple without complex constructs like aggregates, etc., it is sent to all shards.

Vitess can handle more complex queries. For now, refer to the [design doc](https://github.com/vitessio/vitess/blob/main/doc/V3HighLevelDesign.md) for background information on how it handles them.
Vitess can handle more complex queries with the new Gen4 planner.
Copy link
Member Author

Choose a reason for hiding this comment

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

I couldn't find a doc to link for the gen4 planner. The previously linked design doc is for v3, which I don't think is valid anymore.

@netlify
Copy link

netlify bot commented Dec 19, 2022

Deploy Preview for vitess ready!

Name Link
🔨 Latest commit 0d5d2fc
🔍 Latest deploy log https://app.netlify.com/sites/vitess/deploys/63a195f7c5c0220009904b86
😎 Deploy Preview https://deploy-preview-1293--vitess.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.

@GuptaManan100
Copy link
Member Author

This PR also has the fix to the presentations page to fix the width and height of the pdfs there -

Before -
Screenshot 2022-12-16 at 3 01 27 PM

After -
Screenshot 2022-12-20 at 10 50 21 AM

cc - @rsajwani


Thus, you can survive sudden primary failure without losing any transactions that were reported to clients as completed. In MySQL 5.7+, this guarantee is strengthened slightly to preventing loss of any transactions that were ever **committed** on the original primary, eliminating so-called [phantom reads](http://bugs.mysql.com/bug.php?id=62174).

On the other hand these behaviors also give a requirement that each shard must have at least 2 tablets with type *replica* (with addition of the primary that can be demoted to type *replica* this gives a minimum of 3 tablets with initial type *replica*). This will allow for the primary to have a semi-sync acker when one of the replica tablets is down for any reason (for a version update, machine reboot, schema swap or anything else).
These requirements will changed based on the durability policy.
Copy link
Member

Choose a reason for hiding this comment

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

will changed -> will change

Copy link
Collaborator

@rsajwani rsajwani left a comment

Choose a reason for hiding this comment

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

LGTM

@GuptaManan100 GuptaManan100 merged commit a786a1b into vitessio:prod Jan 5, 2023
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