Peter, the LegacyReplicationPolicy class is described in the existing
KIP-382 and is a requirement for the deprecation of MM1. I was planning to
implement it but would love the help if you're interested.
Ryanne
On Tue, Jan 21, 2020, 8:25 AM Péter Sinóros-Szabó
<peter.sinoros-sz...@transferwise.com.invalid> wrote:
> Ryanne,
>
> I didn't do much work yet, just checked the Interface to see if it is easy
> to implement or not.
>
> > The PR for LegacyReplicationPolicy should include any relevant fixes to
> get it to run without crashing
> Do you mean that there is already a PR for LegacyReplicationPolicy? If
> there is, please link it here, I could not find it.
>
> Thanks,
> Peter
>
>
>
>
> On Fri, 17 Jan 2020 at 20:58, Ryanne Dolan <ryannedo...@gmail.com> wrote:
>
> > Peter, KIP-382 includes LegacyReplicationPolicy for this purpose, but no,
> > it has not been implemented yet. If you are interested in writing the PR,
> > it would not require a separate KIP before merging. Looks like you are
> > already doing the work :)
> >
> > It is possible, as you point out, that returning nulls like that will
> break
> > parts of MM2. The PR for LegacyReplicationPolicy should include any
> > relevant fixes to get it to run without crashing.
> >
> > Certainly some parts of MM2 will not work as intended when used with
> > LegacyReplicationPolicy, e.g. MM2 would not be able to prevent cycles.
> But
> > of course it should still _run_ and should work roughly the same as MM1.
> >
> > I'm happy to work with you on the PR if you are interested.
> >
> > Ryanne
> >
> > On Fri, Jan 17, 2020 at 9:34 AM Péter Sinóros-Szabó
> > <peter.sinoros-sz...@transferwise.com.invalid> wrote:
> >
> > > Hi Sebastian & Ryanne,
> > >
> > > do you have maybe an implementation of this is just some ideas about
> how
> > to
> > > implement the policy that does not rename topics?
> > > I am checking the ReplicationPolicy interface and don't really know
> what
> > > the impact will be if I implement this:
> > >
> > > public String formatRemoteTopic(String sourceClusterAlias, String
> topic)
> > {
> > > return topic; }
> > > public String topicSource(String topic) { return null; // well, I do
> not
> > > really know if this were a mirrored topic or not }
> > > public String upstreamTopic(String topic) { return topic; // well, I
> do
> > > not really know if this were a mirrored topic or not}
> > > public String originalTopic(String topic) { return topic; }
> > >
> > > Thanks,
> > > Peter
> > >
> > > On Mon, 30 Dec 2019 at 06:57, Ryanne Dolan <ryannedo...@gmail.com>
> > wrote:
> > >
> > > > Sebastian, you can drop in a custom jar in the "Connect plug-in path"
> > and
> > > > MM2 will be able to load it. That enables you to implement your own
> > > > ReplicationPolicy (and other pluggable interfaces) without compiling
> > > > everything.
> > > >
> > > > In an upcoming release we'll have a "LegacyReplicationPolicy" that
> does
> > > not
> > > > rename topics. It's possible "SimpleReplicationPolicy" is a better
> > name.
> > > >
> > > > Be advised that some features depend on correct ReplicationPolicy
> > > > semantics, which LegacyReplicationPolicy will explicitly break. For
> > > > example, MM2 cannot prevent cycles if topics are not renamed (or some
> > > other
> > > > similar mechanism is used).
> > > >
> > > > Ryanne
> > > >
> > > > On Sun, Dec 29, 2019, 7:41 PM Sebastian Schmitz <
> > > > sebastian.schm...@propellerhead.co.nz> wrote:
> > > >
> > > > > Hello,
> > > > >
> > > > > I found that it's using the DefaultReplicationPolicy that always
> > > returns
> > > > > "sourceClusterAlias + separator + topic" with only the separator
> > being
> > > > > configurable in the configuration-file with
> > > REPLICATION_POLICY_SEPARATOR.
> > > > >
> > > > > It seems like I need a different ReplicationPolicy, like a
> > > > > SimpleReplicationPolicy which always returns "topic" for the
> > > > > formatRemoteTopic, then. But that would mean that I can't download
> > the
> > > > > Binaries and have to build the whole thing myself after adding the
> > new
> > > > > Policy-file!?
> > > > > Or I could create a PR for a SimpleReplicationPolicy to be in some
> > > > > future build...
> > > > >
> > > > > Any suggestions for this?
> > > > >
> > > > > Thanks
> > > > >
> > > > > Sebastian
> > > > >
> > > > >
> > > > > On 30-Dec-19 1:39 PM, Sebastian Schmitz wrote:
> > > > > > Hello,
> > > > > >
> > > > > > another thing I found and didn't find any configuration in the
> KIP
> > > yet
> > > > > > was that if I have two clusters (source and target) and a topic
> > > > > > "replicateme" on the source-cluster it will get replicated to the
> > > > > > target-cluster as "source.replicateme".
> > > > > >
> > > > > > How can I stop it from adding the cluster-name in front of the
> > > > > > topic-name on target-cluster?
> > > > > >
> > > > > > Thanks
> > > > > >
> > > > > > Sebastian
> > > > > >
> > > > > > On 27-Dec-19 7:24 AM, Sebastian Schmitz wrote:
> > > > > >> Hello Ryanne,
> > > > > >>
> > > > > >> Is there a way to prevent that from happening? We have two
> > separate
> > > > > >> clusters with some topics being replicated to the second one for
> > > > > >> reporting. If we replicate everything again that reporting would
> > > > > >> probably have some problems.
> > > > > >>
> > > > > >> Yes, I wondered when the Networking-guys would come and complain
> > > > > >> about me using too much bandwidth on the VPN-Link ;)
> > > > > >>
> > > > > >> Thanks
> > > > > >>
> > > > > >> Sebastian
> > > > > >>
> > > > > >> On 24-Dec-19 1:11 PM, Ryanne Dolan wrote:
> > > > > >>> Glad to hear you are replicating now :)
> > > > > >>>
> > > > > >>>> it probably started mirroring the last seven days as there was
> > no
> > > > > >>>> offset
> > > > > >>> for the new consumer-group.
> > > > > >>>
> > > > > >>> That's correct -- MM2 will replicate the entire topic, as far
> > back
> > > > > >>> as the
> > > > > >>> retention period. However, technically there are no consumer
> > groups
> > > > > >>> in MM2!
> > > > > >>>
> > > > > >>> 550MB/s in a test cluster sounds pretty good to me. Try
> > increasing
> > > > > >>> "tasks.max" and adding additional nodes.
> > > > > >>>
> > > > > >>> Ryanne
> > > > > >>>
> > > > > >>>
> > > > > >>> On Mon, Dec 23, 2019 at 5:40 PM Sebastian Schmitz <
> > > > > >>> sebastian.schm...@propellerhead.co.nz> wrote:
> > > > > >>>
> > > > > >>>> Hello again!
> > > > > >>>>
> > > > > >>>> Some probably important configs I found out:
> > > > > >>>>
> > > > > >>>> We need this to enable mirroring as it seems to disabled by
> > > default?
> > > > > >>>>
> > > > > >>>> source->target.enabled = true
> > > > > >>>> target->source.enabled = true
> > > > > >>>>
> > > > > >>>> Also, the Client-IDs can be configured using:
> > > > > >>>>
> > > > > >>>> source.client.id = my_cool_id
> > > > > >>>> target.client.id = my_cooler_id
> > > > > >>>>
> > > > > >>>> I configured them to include the ID of the server and the name
> > of
> > > > the
> > > > > >>>> environment to have separate IDs per mirror-node.
> > > > > >>>>
> > > > > >>>> After adding these two, it looks a bit better than before, but
> > > > > >>>> still not
> > > > > >>>> satisfied as it started to mirror from my prod to test with
> > > 550MB/s
> > > > as
> > > > > >>>> it probably started mirroring the last seven days as there was
> > no
> > > > > >>>> offset
> > > > > >>>> for the new consumer-group. That's next on my list to solve.
> > > > > >>>>
> > > > > >>>> Best regards
> > > > > >>>>
> > > > > >>>> Sebastian
> > > > > >>>>
> > > > > >>>> On 24-Dec-19 8:34 AM, Sebastian Schmitz wrote:
> > > > > >>>>> Hello,
> > > > > >>>>>
> > > > > >>>>> I tried running this connect-mirror-config:
> > > > > >>>>>
> > > > > >>>>> <snip>
> > > > > >>>>> name = $MIRROR_NAME
> > > > > >>>>> clusters = source, target
> > > > > >>>>> source.bootstrap.servers = $SOURCE_SERVERS
> > > > > >>>>> target.bootstrap.servers = $TARGET_SERVERS
> > > > > >>>>> source->target.topics = $SOURCE_TARGET_TOPICS
> > > > > >>>>> target->source.topics = $TARGET_SOURCE_TOPICS
> > > > > >>>>> source->target.emit.heartbeats.enabled = true
> > > > > >>>>> target->source.emit.heartbeats.enabled = true
> > > > > >>>>> connector.class =
> > > > > >>>>> org.apache.kafka.connect.mirror.MirrorSourceConnector
> > > > > >>>>>
> > > > > >>>>> # disable some new features
> > > > > >>>>> refresh.topics.enabled = false
> > > > > >>>>> refresh.groups.enabled = false
> > > > > >>>>> emit.checkpoints.enables = true
> > > > > >>>>> emit.heartbeats.enabled = true
> > > > > >>>>> sync.topic.configs.enabled = false
> > > > > >>>>> sync.topic.acls.enabled = false
> > > > > >>>>> </snip>
> > > > > >>>>>
> > > > > >>>>> SOURCE_SERVERS and TARGET_SERVERS are a comma-separated list
> of
> > > > three
> > > > > >>>>> brokers with ports.
> > > > > >>>>> The TOPICS are |-separated lists of topics.
> > > > > >>>>>
> > > > > >>>>> I get these warning during startup which is a bit weird as I
> > > never
> > > > > >>>>> supplied any of those settings, but maybe I should?
> > > > > >>>>>
> > > > > >>>>> [2019-12-23 00:36:25,918] WARN The configuration
> > > > > >>>>> 'config.storage.topic' was supplied but isn't a known config.
> > > > > >>>>> (org.apache.kafka.clients.producer.ProducerConfig:355)
> > > > > >>>>> [2019-12-23 00:36:25,918] WARN The configuration
> > > > > >>>>> 'producer.bootstrap.servers' was supplied but isn't a known
> > > config.
> > > > > >>>>> (org.apache.kafka.clients.producer.ProducerConfig:355)
> > > > > >>>>> [2019-12-23 00:36:25,918] WARN The configuration 'group.id'
> > was
> > > > > >>>>> supplied but isn't a known config.
> > > > > >>>>> (org.apache.kafka.clients.producer.ProducerConfig:355)
> > > > > >>>>> [2019-12-23 00:36:25,919] WARN The configuration
> > > > > >>>>> 'status.storage.topic' was supplied but isn't a known config.
> > > > > >>>>> (org.apache.kafka.clients.producer.ProducerConfig:355)
> > > > > >>>>> [2019-12-23 00:36:25,919] WARN The configuration
> > > 'header.converter'
> > > > > >>>>> was supplied but isn't a known config.
> > > > > >>>>> (org.apache.kafka.clients.producer.ProducerConfig:355)
> > > > > >>>>> [2019-12-23 00:36:25,919] WARN The configuration
> > > > > >>>>> 'consumer.bootstrap.servers' was supplied but isn't a known
> > > config.
> > > > > >>>>> (org.apache.kafka.clients.producer.ProducerConfig:355)
> > > > > >>>>> [2019-12-23 00:36:25,919] WARN The configuration
> > > > > >>>>> 'offset.storage.topic' was supplied but isn't a known config.
> > > > > >>>>> (org.apache.kafka.clients.producer.ProducerConfig:355)
> > > > > >>>>> [2019-12-23 00:36:25,919] WARN The configuration
> > > 'value.converter'
> > > > > >>>>> was
> > > > > >>>>> supplied but isn't a known config.
> > > > > >>>>> (org.apache.kafka.clients.producer.ProducerConfig:355)
> > > > > >>>>> [2019-12-23 00:36:25,919] WARN The configuration
> > 'key.converter'
> > > > was
> > > > > >>>>> supplied but isn't a known config.
> > > > > >>>>> (org.apache.kafka.clients.producer.ProducerConfig:355)
> > > > > >>>>> [2019-12-23 00:36:25,919] WARN The configuration
> > > > > >>>>> 'admin.bootstrap.servers' was supplied but isn't a known
> > config.
> > > > > >>>>> (org.apache.kafka.clients.producer.ProducerConfig:355)
> > > > > >>>>>
> > > > > >>>>> And this error:
> > > > > >>>>>
> > > > > >>>>> [2019-12-23 00:36:29,320] ERROR Plugin class loader for
> > > connector:
> > > > > >>>>> 'org.apache.kafka.connect.mirror.MirrorSourceConnector' was
> not
> > > > > >>>>> found.
> > > > > >>>>> Returning:
> > > > > >>>>>
> > > > >
> > >
> org.apache.kafka.connect.runtime.isolation.DelegatingClassLoader@5c316230
> > > > > >>>>>
> > > > > >>>>>
> > > > >
> > (org.apache.kafka.connect.runtime.isolation.DelegatingClassLoader:165)
> > > > > >>>>>
> > > > > >>>>>
> > > > > >>>>> First I tried the config mentioned in the KIP for
> "MirrorMaker
> > > > > >>>>> Clusters" which didn't work and I found removing the
> "cluster."
> > > > from
> > > > > >>>>> the bootstrap-servers made it work a bit more, at least it
> > didn't
> > > > > >>>>> complain about not having any servers in the config.
> > > > > >>>>> So, I checked the "Running a dedicated MirrorMaker
> cluster"from
> > > the
> > > > > >>>>> KIP, which is basically more or less the same, but without
> the
> > > > > >>>>> "cluster." for the servers and it does at least start and it
> > > looks
> > > > > >>>>> like all the three MMs find each other, but no mirroring
> taking
> > > > > >>>>> place.
> > > > > >>>>>
> > > > > >>>>> Running the legacy-config from the old MM is working fine
> > though.
> > > > > >>>>> I'll
> > > > > >>>>> try to do some more digging today, so if you need some of
> those
> > > > very
> > > > > >>>>> verbose logs or something else just let me know. I am sure
> > that I
> > > > can
> > > > > >>>>> figure this out and just wanted to know if the documentation
> > will
> > > > get
> > > > > >>>>> extended as the new MM2 has a lot of features and is a bit
> more
> > > > > >>>>> complicated than the old one...
> > > > > >>>>>
> > > > > >>>>> Thanks
> > > > > >>>>>
> > > > > >>>>> Sebastian
> > > > > >>>>>
> > > > > >>>>> On 24-Dec-19 8:06 AM, Ryanne Dolan wrote:
> > > > > >>>>>> Hello Sebastian, please let us know what issues you are
> facing
> > > > > >>>>>> and we
> > > > > >>>>>> can
> > > > > >>>>>> probably help. Which config from the KIP are you
> referencing?
> > > > > >>>>>> Also check
> > > > > >>>>>> out the readme under ./connect/mirror for more examples.
> > > > > >>>>>>
> > > > > >>>>>> Ryanne
> > > > > >>>>>>
> > > > > >>>>>> On Mon, Dec 23, 2019, 12:58 PM Sebastian Schmitz <
> > > > > >>>>>> sebastian.schm...@propellerhead.co.nz> wrote:
> > > > > >>>>>>
> > > > > >>>>>>> Hello,
> > > > > >>>>>>>
> > > > > >>>>>>> I'm currently trying to implement the new Kafka 2.4.0 and
> the
> > > > > >>>>>>> new MM2.
> > > > > >>>>>>>
> > > > > >>>>>>> However, it looks like the only documentation available is
> > the
> > > > > >>>>>>> KIP-382,
> > > > > >>>>>>> and the documentation
> > > > > >>>>>>> (
> > > https://kafka.apache.org/documentation/#basic_ops_mirror_maker)
> > > > > >>>>>>> for
> > > > > >>>>>>> the
> > > > > >>>>>>> MM isn't yet updated, and the documentation in the KIP
> seems
> > to
> > > > be
> > > > > >>>>>>> missing some stuff as I get a lot of errors and warning
> when
> > > > > >>>>>>> starting
> > > > > >>>>>>> the MM2 as connect-mirror, and it doesn't mirror, so I
> > probably
> > > > > >>>>>>> have
> > > > > >>>>>>> some mistakes in my configuration, but can't confirm this
> as
> > > > > >>>>>>> it's the
> > > > > >>>>>>> same as in the KIP.
> > > > > >>>>>>>
> > > > > >>>>>>> Any plans when the documentation will be updated?
> > > > > >>>>>>>
> > > > > >>>>>>> Thanks
> > > > > >>>>>>>
> > > > > >>>>>>> Sebastian
> > > > > >>>>>>>
> > > > > >>>>>>>
> > > > > >>>>>>> --
> > > > > >>>>>>> DISCLAIMER
> > > > > >>>>>>> This email contains information that is confidential and
> > which
> > > > > >>>>>>> may be
> > > > > >>>>>>> legally privileged. If you have received this email in
> error
> > > > please
> > > > > >>>>>>>
> > > > > >>>>>>> notify the sender immediately and delete the email.
> > > > > >>>>>>> This email is intended
> > > > > >>>>>>> solely for the use of the intended recipient and you may
> not
> > > use
> > > > or
> > > > > >>>>>>> disclose this email in any way.
> > > > > >>>>>>>
> > > > > >>>> --
> > > > > >>>> DISCLAIMER
> > > > > >>>> This email contains information that is confidential and which
> > > > > >>>> may be
> > > > > >>>> legally privileged. If you have received this email in error
> > > please
> > > > > >>>>
> > > > > >>>> notify the sender immediately and delete the email.
> > > > > >>>> This email is intended
> > > > > >>>> solely for the use of the intended recipient and you may not
> use
> > > or
> > > > > >>>> disclose this email in any way.
> > > > > >>>>
> > > > >
> > > > > --
> > > > > DISCLAIMER
> > > > > This email contains information that is confidential and which
> > > > > may be
> > > > > legally privileged. If you have received this email in error please
> > > > >
> > > > > notify the sender immediately and delete the email.
> > > > > This email is intended
> > > > > solely for the use of the intended recipient and you may not use or
> > > > > disclose this email in any way.
> > > > >
> > > >
> > >
> >
>
