[jira] [Commented] (KAFKA-8544) Remove legacy kafka.admin.AdminClient

2019-06-21 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on KAFKA-8544:
---

ijuma commented on pull request #6947: KAFKA-8544: Remove legacy 
kafka.admin.AdminClient
URL: https://github.com/apache/kafka/pull/6947
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Remove legacy kafka.admin.AdminClient
> -
>
> Key: KAFKA-8544
> URL: https://issues.apache.org/jira/browse/KAFKA-8544
> Project: Kafka
>  Issue Type: Improvement
>Reporter: Ismael Juma
>Assignee: Ismael Juma
>Priority: Major
> Fix For: 2.4.0
>
>
> It has been deprecated since 0.11.0, it was never meant as a publicly
> supported API and people should use
> `org.apache.kafka.clients.admin.AdminClient` instead. Its presence
> causes confusion and people still use them accidentally at times.
> `BrokerApiVersionsCommand` uses one method that is not available
> in `org.apache.kafka.clients.admin.AdminClient`, we inline it for now.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (KAFKA-8544) Remove legacy kafka.admin.AdminClient

2019-06-16 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on KAFKA-8544:
---

ijuma commented on pull request #6947: KAFKA-8544: Remove legacy 
kafka.admin.AdminClient
URL: https://github.com/apache/kafka/pull/6947
 
 
   It has been deprecated since 0.11.0, it was never meant as a publicly
   supported API and people should use
   `org.apache.kafka.clients.admin.AdminClient` instead. Its presence
   causes confusion and people still use them accidentally at times.
   
   `BrokerApiVersionsCommand` uses one method that is not available
   in `org.apache.kafka.clients.admin.AdminClient`, we inline it for now.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Remove legacy kafka.admin.AdminClient
> -
>
> Key: KAFKA-8544
> URL: https://issues.apache.org/jira/browse/KAFKA-8544
> Project: Kafka
>  Issue Type: New Feature
>Reporter: Ismael Juma
>Assignee: Ismael Juma
>Priority: Major
> Fix For: 2.4.0
>
>
> It has been deprecated since 0.11.0, it was never meant as a publicly
> supported API and people should use
> `org.apache.kafka.clients.admin.AdminClient` instead. Its presence
> causes confusion and people still use them accidentally at times.
> `BrokerApiVersionsCommand` uses one method that is not available
> in `org.apache.kafka.clients.admin.AdminClient`, we inline it for now.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)