> On March 3, 2017, 3:59 p.m., Di Li wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java > > Lines 4324 (patched) > > <https://reviews.apache.org/r/57168/diff/3/?file=1653055#file1653055line4324> > > > > Why did you need to restore the keys ? stringPropertyNames already > > returns a Set object that you may use directly.
Yes...I can use set directly, I have updated the patch > On March 3, 2017, 3:59 p.m., Di Li wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java > > Lines 4327 (patched) > > <https://reviews.apache.org/r/57168/diff/3/?file=1653055#file1653055line4327> > > > > close instream in finally{} Fixed in new patch > On March 3, 2017, 3:59 p.m., Di Li wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java > > Lines 4329 (patched) > > <https://reviews.apache.org/r/57168/diff/3/?file=1653055#file1653055line4329> > > > > How did Ambari server react to an i/o exception on the blacklist file? > > Please make sure Ambari server start restarts successfully even when the > > blacklist file causes an i/o exception as I don't think it's a sev1 err > > like some others in this class, i.e. can't read db pwd, can't read ssl cert > > pwd. Throwing the exception would cause server error, so I have changed it to print a message in the log > On March 3, 2017, 3:59 p.m., Di Li wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/RootServiceResponseFactory.java > > Lines 168 (patched) > > <https://reviews.apache.org/r/57168/diff/3/?file=1653056#file1653056line168> > > > > The second "tab" seemed to be a real tab than 2 white spaces. please > > make sure all tabs are replaced by spaces. You can set that in Eclipse as a > > general setting. Fixed in new patch - Anita ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57168/#review167838 ----------------------------------------------------------- On March 3, 2017, 8:47 p.m., Anita Jebaraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57168/ > ----------------------------------------------------------- > > (Updated March 3, 2017, 8:47 p.m.) > > > Review request for Ambari, Di Li, Oleksandr Diachenko, Sangeeta Ravindran, > and Vitalyi Brodetskyi. > > > Bugs: AMBARI-20243 > https://issues.apache.org/jira/browse/AMBARI-20243 > > > Repository: ambari > > > Description > ------- > > Currently all the details from the ambari.properties file is being returned > by the API call. > > Some of those information may not be utilized and hence an option can be > provided to filter the properties > > A ambari-blacklist.properties file can be created, the properties that are > entered in the file, will be removed from the api call that returns the > ambari.properties. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java > eaecf35 > > ambari-server/src/main/java/org/apache/ambari/server/controller/RootServiceResponseFactory.java > dadcf09 > > ambari-server/src/test/java/org/apache/ambari/server/configuration/ConfigurationTest.java > 51114f8 > > > Diff: https://reviews.apache.org/r/57168/diff/4/ > > > Testing > ------- > > Added 1 test case > Ran mvn test > > > Thanks, > > Anita Jebaraj > >
