----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52737/ -----------------------------------------------------------
Review request for oozie, András Piros, Attila Sasvari, Peter Cseh, and Robert Kanter. Repository: oozie-git Description ------- This review contains a suggested approach for refactoring LauncherAM on the OYA branch. The main idea is to inject all dependencies of the class via the constructor. In some cases, this requires factory objects. Note that this is necessary because in unit tests, we don't want to create actual resources such as UGI or async ResourceManager callback. In the tests, we can instantiate this class with mocks. The testability is greatly enhanced because various checks and verifications can be performed on the mocks. Tests are currently not in this review (see https://issues.apache.org/jira/browse/OOZIE-2685). Diffs ----- core/src/test/java/org/apache/oozie/action/hadoop/TestLauncherAM.java ed29299 sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/AMRMCallBackHandler.java PRE-CREATION sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/AMRMClientAsyncFactory.java PRE-CREATION sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/ErrorHolder.java PRE-CREATION sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/HdfsAndLocalFSOperations.java PRE-CREATION sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherAM.java 85d78c6 sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherAMCallbackNotifier.java 23648b8 sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherAMCallbackNotifierFactory.java PRE-CREATION sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/PrepareActionsDriver.java 4a51d48 sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/PrepareActionsHandler.java PRE-CREATION sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/SequenceFileWriterFactory.java PRE-CREATION Diff: https://reviews.apache.org/r/52737/diff/ Testing ------- Unit tests are being written right now. Thanks, Peter Bacsko