> On 五月 10, 2017, 5:54 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1/package/scripts/service_check.py > > Line 44 (original), 44 (patched) > > <https://reviews.apache.org/r/57580/diff/1/?file=1662958#file1662958line44> > > > > Please make the same changes for > > ambari-server/src/main/resources/common-services/KAFKA/0.10.0.3.0
Please see the new attached file AMBARI-20434-1.patch ,thanks - Wang ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57580/#review174515 ----------------------------------------------------------- On 五月 11, 2017, 2:11 a.m., Wang Jie wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57580/ > ----------------------------------------------------------- > > (Updated 五月 11, 2017, 2:11 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Andrii Tkach, Dmitro > Lisnichenko, Jaimin Jetly, Jayush Luniya, and Srimanth Gunturi. > > > Bugs: AMBARI-20434 > https://issues.apache.org/jira/browse/AMBARI-20434 > > > Repository: ambari > > > Description > ------- > > Problem: > Kafka Service Check will create the kafka topic only if the topic doesn't > exist. > if it exists, Kafka Service Check will return success directly. > After topic is being created, we sense that each time running service check > will not reflect the kafka actual state correctly > > Suggestion: > Every running kafka service check, delete the existed topic and recreate it. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1/package/scripts/service_check.py > 0f3a417 > > ambari-server/src/main/resources/common-services/KAFKA/0.9.0/configuration/kafka-broker.xml > 0275358 > > > Diff: https://reviews.apache.org/r/57580/diff/1/ > > > Testing > ------- > > Done > > > File Attachments > ---------------- > > AMBARI-20434-1.patch > > https://reviews.apache.org/media/uploaded/files/2017/05/11/299ea8d7-8093-42d9-8a64-ceadcfbea90a__AMBARI-20434-1.patch > > > Thanks, > > Wang Jie > >
