----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/#review148703 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/audit/request/eventcreator/RequestEventCreator.java (line 90) <https://reviews.apache.org/r/51843/#comment216206> Please add a unit test for this change. - Sebastian Toader On Sept. 13, 2016, 2:25 p.m., wang yaoxin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51843/ > ----------------------------------------------------------- > > (Updated Sept. 13, 2016, 2:25 p.m.) > > > Review request for Ambari, Jonathan Hurley, 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 > > Diff: https://reviews.apache.org/r/51843/diff/ > > > Testing > ------- > > done! > > > Thanks, > > wang yaoxin > >