Re: [DISCUSS] KIP-828: Add the corresponding validator to the configuration where the validator is missing

2022-04-13 Thread Chris Egerton
Hi Riven, I think it may be possible to accomplish most or all of this without a KIP. RE ConfigDef.NonNullAndEmptyString: is there a case that this validator satisfies that isn't satisfied by the existing ConfigDef.NonEmptyString validator [1] ? If a property is required but a user doesn't specif

Re: [DISCUSS] KIP-828: Add the corresponding validator to the configuration where the validator is missing

2022-04-13 Thread Lolov, Christo
Hello! I am struggling to understand the context of the suggested change. Is the point to add dynamic validation? In other words, is the idea that when a person adds another SASL module it would automatically be picked up by the validation framework and the validation list would automatically c

[DISCUSS] KIP-828: Add the corresponding validator to the configuration where the validator is missing

2022-04-08 Thread Riven Sun
Hi devs, I've created a KIP that aims to add the corresponding validator to the config where the validator is missing. In order for the program to detect these incorrect configurations during initialization: https://cwiki.apache.org/confluence/display/KAFKA/KIP-828%3A+Add+the+corresponding+validat