[ 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