Hi Mickael, Hi all, Thanks for the feedback! I have adapted the KIP description - actually much shorter and just reflecting the general functionality and interface/configuration changes.
Kindly let me know if you have any comments, questions, or suggestions for this KIP! Thanks, Roman Am Fr., 5. Jan. 2024 um 17:36 Uhr schrieb Mickael Maison < mickael.mai...@gmail.com>: > Hi Roman, > > Thanks for the KIP! This would be a useful improvement. > > Ideally you want to make a concerte proposal in the KIP instead of > listing a series of options. Currently the KIP seems to list two > alternatives. > > Also a KIP focuses on the API changes rather than on the pure > implementation. It seems you're proposing adding a configuration to > the DropHeaders SMT. It would be good to describe that new > configuration. For example see KIP-911 which also added a > configuration. > > Thanks, > Mickael > > On Mon, Oct 16, 2023 at 9:50 AM Roman Schmitz <roman.schm...@gmail.com> > wrote: > > > > Hi Andrew, > > > > Ok, thanks for the feedback! I added a few more details and code examples > > to explain the proposed changes. > > > > Thanks, > > Roman > > > > Am So., 15. Okt. 2023 um 22:12 Uhr schrieb Andrew Schofield < > > andrew_schofield_j...@outlook.com>: > > > > > Hi Roman, > > > Thanks for the KIP. I think it’s an interesting idea, but I think the > KIP > > > document needs some > > > more details added before it’s ready for review. For example, here’s a > KIP > > > in the same > > > area which was delivered in an earlier version of Kafka. I think this > is a > > > good KIP to copy > > > for a suitable level of detail and description ( > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-585%3A+Filter+and+Conditional+SMTs > > > ). > > > > > > Hope this helps. > > > > > > Thanks, > > > Andrew > > > > > > > On 15 Oct 2023, at 21:02, Roman Schmitz <roman.schm...@gmail.com> > wrote: > > > > > > > > Hi all, > > > > > > > > While working with different customers I came across the case several > > > times > > > > that we'd like to not only explicitly remove headers by name but by > > > pattern > > > > / regexp. Here is a KIP for this feature! > > > > > > > > Please let me know if you have any comments, questions, or > suggestions! > > > > > > > > https://cwiki.apache.org/confluence/x/oYtEE > > > > > > > > Thanks, > > > > Roman > > > > > > >