kafka-jira mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Guozhang Wang (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (KAFKA-4857) Replace StreamsKafkaClient with AdminClient in Kafka Streams
Date Mon, 13 Nov 2017 20:47:01 GMT

     [ https://issues.apache.org/jira/browse/KAFKA-4857?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Guozhang Wang updated KAFKA-4857:
---------------------------------
    Description: 
Streams uses {{KafkaClientSupplier}} to get consumer/restore-consumer/producer clients. Streams
also uses one more client for admin purpose namely {{StreamsKafkaClient}} that is instantiated
"manually".

With the newly upcoming {{AdminClient}} from KIP-117, we can simplify (or even replace {{StreamsKafkaClient}}
with the new {{AdminClient}}. We furthermore want to unify how the client in generated and
extend {{KafkaClientSupplier}} with method that return this client.

NOTE: The public facing changes are summarized in a separate ticket KAFKA-6170, and this ticket
is only for the internal swap, with the accepted criterion to completely remove StreamsKafkaClient
with the newly introduced KafkaAdminClient.

  was:
Streams uses {{KafkaClientSupplier}} to get consumer/restore-consumer/producer clients. Streams
also uses one more client for admin purpose namely {{StreamsKafkaClient}} that is instantiated
"manually".

With the newly upcoming {{AdminClient}} from KIP-117, we can simplify (or even replace {{StreamsKafkaClient}}
with the new {{AdminClient}}. We furthermore want to unify how the client in generated and
extend {{KafkaClientSupplier}} with method that return this client.

As this is a public API change, a KIP is required.


> Replace StreamsKafkaClient with AdminClient in Kafka Streams
> ------------------------------------------------------------
>
>                 Key: KAFKA-4857
>                 URL: https://issues.apache.org/jira/browse/KAFKA-4857
>             Project: Kafka
>          Issue Type: Bug
>          Components: streams
>            Reporter: Matthias J. Sax
>            Assignee: Matthias J. Sax
>              Labels: needs-kip
>
> Streams uses {{KafkaClientSupplier}} to get consumer/restore-consumer/producer clients.
Streams also uses one more client for admin purpose namely {{StreamsKafkaClient}} that is
instantiated "manually".
> With the newly upcoming {{AdminClient}} from KIP-117, we can simplify (or even replace
{{StreamsKafkaClient}} with the new {{AdminClient}}. We furthermore want to unify how the
client in generated and extend {{KafkaClientSupplier}} with method that return this client.
> NOTE: The public facing changes are summarized in a separate ticket KAFKA-6170, and this
ticket is only for the internal swap, with the accepted criterion to completely remove StreamsKafkaClient
with the newly introduced KafkaAdminClient.



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

Mime
View raw message