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
HDDS-7108. Include supported EC data-parity combinations in error message #3665
Conversation
@umamaheswararao @kerneltime Feel free to take a look if the added message helps! Thanks! |
LGTM, please address failing CI tests |
sure! looking |
Thank you, @DaveTeng0 for this patch. |
"Invalid data-parity replication config " + | ||
"for type " + replicationConfig.getReplicationType() + | ||
" and replication " + replication + "." + | ||
" Supported data-parity are 3-2,6-3,10-4"); |
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.
Sorry for the late review.
This exception may be thrown for non-EC replication config, too, in which case data-parity
is not applicable. The message should be different based on replication type.
CLI commands validate RATIS replication factor before reaching this check, but Java code using RpcClient#createKey
directly could encounter this problem.
What changes were proposed in this pull request?
Need proper error message on console output when incorrect replication data-parity config is passed
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7108
How was this patch tested?
Manually test from developer machine