----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/#review149197 -----------------------------------------------------------
Ship it! Could you please paste the unitest result to 'Testing Done' section. Aslo the latest patch seems to be correupted. - Sandor Magyari On Sept. 16, 2016, 12:54 p.m., wang yaoxin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51843/ > ----------------------------------------------------------- > > (Updated Sept. 16, 2016, 12:54 p.m.) > > > Review request for Ambari, Jonathan Hurley, Oliver Szabo, Robert Nettleton, > and Sebastian Toader. > > > Bugs: AMBARI-18374 > https://issues.apache.org/jira/browse/AMBARI-18374 > > > Repository: ambari > > > Description > ------- > > when rolling restart services ( eg:DataNode ) ,cluster name in audit log show > null > > 2016-09-13T17:29:22.033+0800, User(admin), RemoteIp(127.0.0.1), > Operation(Request from server), RequestType(POST), > url(http://localhost:8080/api/v1/clusters/amabri/requests), ResultStatus(202 > Accepted), Command(RESTART), Cluster name(null) > > 2016-09-13T17:29:22.042+0800, User(admin), > Operation(PARSE.ROLLING-RESTART.DATANODE.1.1), Status(IN_PROGRESS), > RequestId(15) > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/audit/request/eventcreator/RequestEventCreator.java > 075e328 > > ambari-server/src/test/java/org/apache/ambari/server/audit/request/creator/RequestEventCreatorTest.java > 1074f0e > > Diff: https://reviews.apache.org/r/51843/diff/ > > > Testing > ------- > > done! > > > Thanks, > > wang yaoxin > >