> On Feb. 14, 2017, 11:02 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1/configuration/kafka-broker.xml, > > line 157 > > <https://reviews.apache.org/r/56677/diff/1/?file=1633533#file1633533line157> > > > > During a fresh install this is correct. > > > > I'm asking what the behavior should be for the property that is being > > deleted during a stack upgrade, EU/RU to this version of Kafka.
During EU/RU, property log.cleanup.interval.mins should be deleted. So,during EU/RU the old property should be deleted and new property should be replaced. - Bharat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56677/#review165588 ----------------------------------------------------------- On Feb. 14, 2017, 7:35 p.m., Bharat Viswanadham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56677/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2017, 7:35 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Juanjo > Marron. > > > Repository: ambari > > > Description > ------- > > Updated to remove deprecated property and updated property in latest release > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1/configuration/kafka-broker.xml > 96b7750 > > Diff: https://reviews.apache.org/r/56677/diff/ > > > Testing > ------- > > > Thanks, > > Bharat Viswanadham > >
