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

Joel Koshy updated KAFKA-668:
-----------------------------

    Attachment: KAFKA-668-v1.patch

This is a pretty straightforward change. It's slightly hacky in that I'm 
appending the :jmxport to the zk string, and it is effectively ignored in the 
Broker class. I preferred this over adding a jmxPort field to the Broker class 
as that would be cause wide-spread edits.
                
> Controlled shutdown admin tool should not require controller JMX url/port to 
> be supplied
> ----------------------------------------------------------------------------------------
>
>                 Key: KAFKA-668
>                 URL: https://issues.apache.org/jira/browse/KAFKA-668
>             Project: Kafka
>          Issue Type: Bug
>    Affects Versions: 0.8, 0.8.1
>            Reporter: Joel Koshy
>             Fix For: 0.8
>
>         Attachments: KAFKA-668-v1.patch
>
>
> The controlled shutdown admin command takes a zookeeper string and also 
> requires the user to supply the controller's jmx url/port. This is a bit 
> annoying since the purpose of the zookeeper string is to discover the 
> controller. The tool should require exactly one of these options. If 
> zookeeper is supplied then discover the controller and its jmx port (which 
> means we will need to add the jmx port information to zk).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to