> On April 13, 2017, 8:26 p.m., Di Li wrote: > > Ship It! > > Bharat Viswanadham wrote: > Hi Di, > Could you please help in commit these changes.
please close this RB request as the change is pushed to trunk as https://git-wip-us.apache.org/repos/asf?p=ambari.git;a=commit;h=dd3fdc220d9ad3b2b723a201ce7aa5d767850a58 now. - Di ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58431/#review171929 ----------------------------------------------------------- On April 13, 2017, 8:11 p.m., Bharat Viswanadham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58431/ > ----------------------------------------------------------- > > (Updated April 13, 2017, 8:11 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Di Li. > > > Bugs: AMBARI-20761 > https://issues.apache.org/jira/browse/AMBARI-20761 > > > Repository: ambari > > > Description > ------- > > Update zookeeper.connect description > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1/configuration/kafka-broker.xml > 96b7750 > > > Diff: https://reviews.apache.org/r/58431/diff/1/ > > > Testing > ------- > > It is a document change. Changed the description. > Tested it with install Kafka and zookeeper with change in the > zookeeper.connect to <<host>>:<<port>>/chroot/path and Kafka Broker is > successfully started. > And tried running console consumer > bin/kafka-console-consumer.sh --zookeeper heel1.fyre.ibm.com:2181/kafka/path > --from-beginning --topic ambari_service_check > This also worked fine. > > > Thanks, > > Bharat Viswanadham > >
