mimaison commented on code in PR #15756: URL: https://github.com/apache/kafka/pull/15756#discussion_r1668173616
########## connect/transforms/src/main/java/org/apache/kafka/connect/transforms/Cast.java: ########## @@ -67,6 +67,7 @@ public abstract class Cast<R extends ConnectRecord<R>> implements Transformation + "or value (<code>" + Value.class.getName() + "</code>)."; public static final String SPEC_CONFIG = "spec"; + public static final String REPLACE_NULL_WITH_DEFAULT = "replace.null.with.default"; Review Comment: In some files we have the `_CONFIG` suffix, for example `Cast` but in some others like `HeadersFrom` we don't. When I asked you to make changes I meant so it's consistent in each file. Sorry if I wasn't clear enough. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org