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
update table size docs #3175
update table size docs #3175
Conversation
Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
😊 Welcome @ramaraochavali! This is either your first contribution to the Istio api repo, or it's been You can learn more about the Istio working groups, Code of Conduct, and contribution guidelines Thanks for contributing! Courtesy of your friendly welcome wagon. |
networking/v1/destination_rule.proto
Outdated
@@ -420,6 +420,8 @@ message LoadBalancerSettings { | |||
// The table size for Maglev hashing. This helps in controlling the | |||
// disruption when the backend hosts change. | |||
// Increasing the table size reduces the amount of disruption. | |||
// The table size must be prime number limited to 5000011. |
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.
Should this maybe be:
// The table size must be prime number limited to 5000011. | |
// The table size must be prime number less than 5000011. |
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.
Is 5000011 legit table size?
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.
Yes. As per envoy docs it is
Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
No description provided.