Hi Ryanne, With respect, there's a difference between "we still use it because we can't be bothered to switch to MM2, or just haven't yet" and "it's important for xyz because MM2 doesn't serve our use case properly". While the former is not a good reason to argue against deprecation, the latter might be, depending on the details. It isn't completely clear to me whether you're asserting that MM2 covers all the same use cases as MM1. On the one hand you don't want to make claims, but on the other you're saying we have MM2 now. An assertion that MM2 addressed all the MM1 use cases would be Ismael's explanation about why MM1 is no longer useful, I think. OTOH the KIP says it is still useful. So personally I'm confused about which the situation is.
Are we deprecating something which for some users MM2 cannot replace? If so, I think the KIP should explain clearly why we're intentionally doing that. Kind regards, Tom On Fri, Mar 26, 2021 at 3:22 PM Ryanne Dolan <ryannedo...@gmail.com> wrote: > Ismael, I think it is very difficult in general to argue for deprecation -- > someone will always say "we still use it" or "it's important for xyz" -- so > I don't want to make claims that prompt such responses. The motivation for > deprecating MM1 is that we now have MM2, and there isn't much else to say, > IMO. > > Ryanne > > On Mon, Mar 22, 2021 at 8:04 AM Ismael Juma <ism...@juma.me.uk> wrote: > > > I am in favor of this change, but the KIP doesn't include proper > > motivation. It says "While the original MirrorMaker remains useful, we > want > > to take advantage of the upcoming 3.0 major release to officially > deprecate > > this legacy code". I would hope we would explain why it's no longer > useful > > instead. > > > > Thanks, > > Ismael > > > > On Sat, Mar 20, 2021 at 10:41 AM Ryanne Dolan <ryannedo...@gmail.com> > > wrote: > > > > > Hey y'all, I'm starting the vote on KIP-720, which proposes to > deprecate > > > the original MirrorMaker in the upcoming 3.0 major release. > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-720%3A+Deprecate+MirrorMaker+v1 > > > > > > Thanks! > > > Ryanne > > > > > >
