----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12932/ -----------------------------------------------------------
(Updated July 29, 2013, 8:02 p.m.) Review request for Sqoop, Abraham Elmahrek, Hari Shreedharan, and Jarek Cecho. Changes ------- Make changes based on Jarcec's suggestions. Thank you, Jarcec. Bugs: SQOOP-1143 https://issues.apache.org/jira/browse/SQOOP-1143 Repository: sqoop-sqoop2 Description (updated) ------- commit 9ebaf0aad64b690c378188d2c53d6dcd0cd30ddb Author: Mengwei Ding <mengwei.d...@gmail.com> Date: Tue Jul 23 16:51:53 2013 -0700 SQOOP-1143 Sqoop2: Provide server audit log for operations upon metadata structures :000000 100644 0000000... 881532d... A core/src/main/java/org/apache/sqoop/audit/AuditLogger.java :000000 100644 0000000... 9e27d7c... A core/src/main/java/org/apache/sqoop/audit/AuditLoggerConstants.java :000000 100644 0000000... 5999e1c... A core/src/main/java/org/apache/sqoop/audit/AuditLoggerError.java :000000 100644 0000000... 2c1075a... A core/src/main/java/org/apache/sqoop/audit/AuditLoggerManager.java :000000 100644 0000000... bfc0d62... A core/src/main/java/org/apache/sqoop/audit/FileAuditLogger.java :100644 100644 90bb327... d4c3b39... M core/src/main/java/org/apache/sqoop/core/SqoopServer.java :100755 100755 76596b4... bb01016... M dist/src/main/server/conf/sqoop.properties :100644 100644 62be8f4... 875729f... M server/src/main/java/org/apache/sqoop/handler/ConnectionRequestHandler.java :100644 100644 4c389cc... d1b6b9a... M server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java :100644 100644 04ffc3c... 616e974... M server/src/main/java/org/apache/sqoop/handler/FrameworkRequestHandler.java :100644 100644 0d15329... 59a4511... M server/src/main/java/org/apache/sqoop/handler/JobRequestHandler.java :100644 100644 65686a8... 6d9777a... M server/src/main/java/org/apache/sqoop/handler/SubmissionRequestHandler.java :100644 100644 f92d107... 64c4536... M server/src/main/java/org/apache/sqoop/handler/VersionRequestHandler.java Diffs (updated) ----- core/src/main/java/org/apache/sqoop/audit/AuditLogger.java PRE-CREATION core/src/main/java/org/apache/sqoop/audit/AuditLoggerConstants.java PRE-CREATION core/src/main/java/org/apache/sqoop/audit/AuditLoggerError.java PRE-CREATION core/src/main/java/org/apache/sqoop/audit/AuditLoggerManager.java PRE-CREATION core/src/main/java/org/apache/sqoop/audit/FileAuditLogger.java PRE-CREATION core/src/main/java/org/apache/sqoop/core/SqoopServer.java 90bb327 dist/src/main/server/conf/sqoop.properties 76596b4 server/src/main/java/org/apache/sqoop/handler/ConnectionRequestHandler.java 62be8f4 server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java 4c389cc server/src/main/java/org/apache/sqoop/handler/FrameworkRequestHandler.java 04ffc3c server/src/main/java/org/apache/sqoop/handler/JobRequestHandler.java 0d15329 server/src/main/java/org/apache/sqoop/handler/SubmissionRequestHandler.java 65686a8 server/src/main/java/org/apache/sqoop/handler/VersionRequestHandler.java f92d107 Diff: https://reviews.apache.org/r/12932/diff/ Testing ------- Do some manual test for new functionalities. Unit test passed. Thanks, Mengwei Ding