----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50742/#review144605 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java (lines 47 - 57) <https://reviews.apache.org/r/50742/#comment210636> Can you provide a description of the use case this patch implements? - Sebastian Toader On Aug. 3, 2016, 12:04 p.m., wang yaoxin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50742/ > ----------------------------------------------------------- > > (Updated Aug. 3, 2016, 12:04 p.m.) > > > Review request for Ambari, Laszlo Puskas, Oliver Szabo, Sandor Magyari, and > Sebastian Toader. > > > Repository: ambari > > > Description > ------- > > OperationStatusAuditEvent and TaskStatusAuditEvent auditlog message should > contain the remoteip info, store in the pg table host_role_command > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionDBAccessorImpl.java > b44dc78 > > ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java > ff2ce92 > > ambari-server/src/main/java/org/apache/ambari/server/audit/event/OperationStatusAuditEvent.java > fd0068e > > ambari-server/src/main/java/org/apache/ambari/server/audit/event/TaskStatusAuditEvent.java > 1682e74 > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java > 6288091 > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 0843616 > ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql > 43bdef9 > > ambari-server/src/test/java/org/apache/ambari/server/audit/OperationStatusAuditEventTest.java > 038cebc > > ambari-server/src/test/java/org/apache/ambari/server/audit/TaskStatusAuditEventTest.java > 8f769c7 > > Diff: https://reviews.apache.org/r/50742/diff/ > > > Testing > ------- > > done > > > Thanks, > > wang yaoxin > >
