Mailing-List: contact jira-help@kafka.apache.org; run by ezmlm
Precedence: bulk
Reply-To: jira@kafka.apache.org
Date: Thu, 27 Jul 2017 20:51:01 +0000 (UTC)
From: =?utf-8?Q?S=C3=B6nke_Liebau_=28JIRA=29?= <jira@apache.org>
To: jira@kafka.apache.org
Message-ID: <JIRA.13085169.1499337274000.30714.1501188661332@Atlassian.JIRA>
In-Reply-To: <JIRA.13085169.1499337274000@Atlassian.JIRA>
References: <JIRA.13085169.1499337274000@Atlassian.JIRA> <JIRA.13085169.1499337274838@jira-lw-us.apache.org>
Subject: [jira] [Assigned] (KAFKA-5563) Clarify handling of connector name
 in config
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
archived-at: Thu, 27 Jul 2017 20:51:07 -0000


     [ https://issues.apache.org/jira/browse/KAFKA-5563?page=3Dcom.atlassia=
n.jira.plugin.system.issuetabpanels:all-tabpanel ]

S=C3=B6nke Liebau reassigned KAFKA-5563:
-----------------------------------

    Assignee: S=C3=B6nke Liebau

> Clarify handling of connector name in config=20
> ---------------------------------------------
>
>                 Key: KAFKA-5563
>                 URL: https://issues.apache.org/jira/browse/KAFKA-5563
>             Project: Kafka
>          Issue Type: Bug
>          Components: KafkaConnect
>    Affects Versions: 0.11.0.0
>            Reporter: S=C3=B6nke Liebau
>            Assignee: S=C3=B6nke Liebau
>            Priority: Minor
>
> The connector name is currently being stored in two places, once at the r=
oot level of the connector and once in the config:
> {code:java}
> {
> =09"name": "test",
> =09"config": {
> =09=09"connector.class": "org.apache.kafka.connect.tools.MockSinkConnecto=
r",
> =09=09"tasks.max": "3",
> =09=09"topics": "test-topic",
> =09=09"name": "test"
> =09},
> =09"tasks": [
> =09=09{
> =09=09=09"connector": "test",
> =09=09=09"task": 0
> =09=09}
> =09]
> }
> {code}
> If no name is provided in the "config" element, then the name from the ro=
ot level is [copied there when the connector is being created|https://githu=
b.com/apache/kafka/blob/trunk/connect/runtime/src/main/java/org/apache/kafk=
a/connect/runtime/rest/resources/ConnectorsResource.java#L95]. If however a=
 name is provided in the config then it is not touched, which means it is p=
ossible to create a connector with a different name at the root level and i=
n the config like this:
> {code:java}
> {
> =09"name": "test1",
> =09"config": {
> =09=09"connector.class": "org.apache.kafka.connect.tools.MockSinkConnecto=
r",
> =09=09"tasks.max": "3",
> =09=09"topics": "test-topic",
> =09=09"name": "differentname"
> =09},
> =09"tasks": [
> =09=09{
> =09=09=09"connector": "test1",
> =09=09=09"task": 0
> =09=09}
> =09]
> }
> {code}
> I am not aware of any issues that this currently causes, but it is at lea=
st confusing and probably not intended behavior and definitely bears potent=
ial for bugs, if different functions take the name from different places.
> Would it make sense to add a check to reject requests that provide differ=
ent names in the request and the config section?


--
This message was sent by Atlassian JIRA
(v6.4.14#64029)