> On Nov. 29, 2016, 12:55 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/java/org/apache/ambari/server/utils/RequestUtils.java, > > line 87 > > <https://reviews.apache.org/r/53943/diff/2/?file=1571575#file1571575line87> > > > > unnecessary space before `;` > > wang yaoxin wrote: > Thanks Attila, I have modified my code
Thanks! - Attila ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53943/#review157224 ----------------------------------------------------------- On Nov. 29, 2016, 1:16 p.m., wang yaoxin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53943/ > ----------------------------------------------------------- > > (Updated Nov. 29, 2016, 1:16 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Oliver > Szabo, Robert Levas, Robert Nettleton, and Sebastian Toader. > > > Bugs: AMBARI-18944 > https://issues.apache.org/jira/browse/AMBARI-18944 > > > Repository: ambari > > > Description > ------- > > Mutil-agent were configured in some production environments, RemoteIp > therefore will obtain mutilple IP addresses, like > (RemoteIp(172.16.0.66,172.16.193.56),)which causes errors in log output. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/utils/RequestUtils.java > 0ac782f > > ambari-server/src/test/java/org/apache/ambari/server/utils/RequestUtilsTest.java > 595127e > > Diff: https://reviews.apache.org/r/53943/diff/ > > > Testing > ------- > > RequestUtilsTest All Tests Passsed ,Process finished with exit code 0 > > > Thanks, > > wang yaoxin > >