Matthias J. Sax

Results 91 comments of Matthias J. Sax

Also, thanks for the PR! Merged to `trunk`.

> There might be some discussion to be had regarding what to say about why these are deprecated. Not sure if we need to put this into the docs? If...

> happy to add those new config in this PR Great. > shall I also add that StreamsConfig.java logic update in this PR, or separate it out since it's a...

Thanks for the PR @Cerchie! Merged to `trunk`.

Thanks @gharris1727 @Cerchie -- can you add fixes into #14448 ?

@s7pandey -- Thanks for this PR -- this is a public API change and thus we need a KIP for it. Not sure if you are familiar with the KIP...

That weird -- signup should just work. (You definitely don't need to file an ICLA or become a committer :)) -- Maybe it's a temporary glitch -- or something is...

Thanks for the PR. Overall LGTM. I am just wondering if we should also update the JavaDocs for all deprecated configs? For example: ``` /** {@code default.windowed.key.serde.inner} */ @SuppressWarnings("WeakerAccess") @Deprecated...

@Cerchie -- this PR would need a rebase as #14360 was merged. Also, seems we introduced a few JavaDoc issues (cf https://github.com/apache/kafka/pull/14360#issuecomment-2108456039) that we should fix in this PR.