-
Notifications
You must be signed in to change notification settings - Fork 13
Clustering page revamp #1270
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
base: main
Are you sure you want to change the base?
Clustering page revamp #1270
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@matea16 being developed, need more time |
|
You have run out of free Bugbot PR reviews for this billing cycle. This will reset on November 12. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
Bug: Ghost Text Haunts File End
Stray text "kube" appears alone at the end of the file, likely accidentally committed during editing or development.
pages/getting-started/install-memgraph/kubernetes.mdx#L405-L406
documentation/pages/getting-started/install-memgraph/kubernetes.mdx
Lines 405 to 406 in 9a7e7de
| kube |
| - `SystemRecoveryRpc` - sent by the MAIN instance to a REPLICA instance for system replication queries (auth, multi-tenancy) | ||
| and other non-graph information | ||
| - `PrepareCommitRpc` - main replici (prva faza u strict syncu ili jedina faza u ostalima), pošalje delta stream | ||
| - `FinalizeCommitRpc` - samo za strict sync za 2pc, s maina na replice ako su sve strict sync replice odgovorile pozitivno za commitat |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Unintended Language in RPC Descriptions
The descriptions for PrepareCommitRpc and FinalizeCommitRpc contain non-English text (appears to be Croatian/Serbian), which looks like accidentally committed development notes. Lines 126-127 contain phrases like "main replici (prva faza u strict syncu ili jedina faza u ostalima), pošalje delta stream" and "samo za strict sync za 2pc, s maina na replice ako su sve strict sync replice odgovorile pozitivno za commitat" instead of proper English documentation.
Release note
Please briefly explain the changes you made which will be added to the changelog.
Related product PRs
PRs from product repo this doc page is related to:
(paste the links to the PRs)
Checklist:
bugfixorfeaturelabel, based on the product PR type you're documentingNote
Major overhaul of clustering docs: new concepts for replication/HA, dedicated Community vs Enterprise guides (setup, best practices, commands), updated navigation, and deployment pages linking to HA Helm/K8s.
pages/clustering.mdxwith clearer Community (replication) vs Enterprise (HA) positioning, callouts, and links; add FAQ.how-replication-works.mdx,how-high-availability-works.mdx,querying-the-cluster-in-high-availability.mdxwith theory, internals, and routing.pages/clustering/replication.mdxand nav_meta.ts.setup-replication-cluster-docker.mdx,setup-replication-cluster-k8s.mdx.best-practices.mdx,replication-commands-reference.mdx, andsystem-replication.mdx(scope clarified).pages/clustering/high-availability.mdxinto hub linking setup/best-practices/commands.setup-ha-cluster-docker.mdx,setup-ha-cluster-docker-compose.mdx,setup-ha-cluster-k8s.mdx,best-practices.mdx,ha-commands-reference.mdx; add nav_meta.ts.pages/clustering/_meta.ts; addpages/clustering/concepts/_meta.ts.deployment/environments/aws.mdxandgetting-started/install-memgraph/kubernetes.mdxto reference HA on EKS/Helm and link to new HA setup page.Written by Cursor Bugbot for commit 9a7e7de. This will update automatically on new commits. Configure here.