----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38942/#review101490 -----------------------------------------------------------
Ship it! lgtm - Abraham Fine On Oct. 5, 2015, 5:23 p.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38942/ > ----------------------------------------------------------- > > (Updated Oct. 5, 2015, 5:23 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2598 > https://issues.apache.org/jira/browse/SQOOP-2598 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > I've made the changes proposed in JIRA. > > > Diffs > ----- > > common/src/main/java/org/apache/sqoop/error/code/AuditLoggerError.java > 1af8d7d > core/src/main/java/org/apache/sqoop/audit/FileAuditLogger.java 49a380c > dist/src/main/conf/sqoop.properties f077ecc > test/src/main/java/org/apache/sqoop/test/minicluster/SqoopMiniCluster.java > 7440025 > > Diff: https://reviews.apache.org/r/38942/diff/ > > > Testing > ------- > > I've valided on real cluster that: > > * We're getting the audit logger as we were before in separate file > * We see all the classes in the main log file > > > Thanks, > > Jarek Cecho > >
