Hi Proven, Thanks for the changes to KIP-900. It looks good to me in general. Here are some suggestions and questions.
1. In the KIP you give the following example: --add-scram SCRAM-SHA-256=[user=alice,password=alice-secret] Is "SCRAM-" required as a prefix? The flag already has the suffix "-scram". Can the value to the flag be SHA-256=[user=alice,password=alice-secret]? 2. Should the CLI document all possible values for the --add-scram? Is SCRAM-SHA-256 the only supported algorithm? 3. Should the short version of the flag --add-scram be -s? I suspect that in the future we may want to add more options like --add-acl and --add-config. Thanks! -- -José
