kafka-jira mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrey Dyachkov (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (KAFKA-5723) Refactor BrokerApiVersionsCommand to use the new AdminClient
Date Sun, 13 Aug 2017 21:04:00 GMT

    [ https://issues.apache.org/jira/browse/KAFKA-5723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16125045#comment-16125045
] 

Andrey Dyachkov edited comment on KAFKA-5723 at 8/13/17 9:03 PM:
-----------------------------------------------------------------

[~viktorsomogyi] am I right that this change does not require KIP? It is not an issue for
me to make it with some help of course. And the second thing about implementation:
I plan to extend AdminClient abstract class to have a method for sending ApiVersionsRequest
and getting the list of nodes calling AdminClient.describeCluster method for node provider
in KafkaAdminClient. Does it make sense at all?


was (Author: andrey.dyachkov@gmail.com):
[~viktorsomogyi] am I right that this change does not require KIP? It is not an issue for
me to make it with some help of course. And the second thing about impl:
I plan to extend AdminClient abstract class to have a method for sending ApiVersionsRequest
and getting the list of nodes calling AdminClient.describeCluster method for node provider
in KafkaAdminClient. Does it make sense at all?

> Refactor BrokerApiVersionsCommand to use the new AdminClient
> ------------------------------------------------------------
>
>                 Key: KAFKA-5723
>                 URL: https://issues.apache.org/jira/browse/KAFKA-5723
>             Project: Kafka
>          Issue Type: Improvement
>            Reporter: Viktor Somogyi
>            Assignee: Viktor Somogyi
>
> Currently it uses the deprecated AdminClient and in order to remove usages of that client,
this class needs to be refactored.



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

Mime
View raw message