You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by Kinga Marton via Review Board <no...@reviews.apache.org> on 2019/07/01 09:04:15 UTC
Re: Review Request 70964: OOZIE-3513 Migrate from
Preconditions.checkNotNull and ParamChecker.notNull
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70964/#review216267
-----------------------------------------------------------
Ship it!
Ship It!
- Kinga Marton
On June 28, 2019, 8:57 a.m., Andras Salamon wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/70964/
> -----------------------------------------------------------
>
> (Updated June 28, 2019, 8:57 a.m.)
>
>
> Review request for oozie and Kinga Marton.
>
>
> Repository: oozie-git
>
>
> Description
> -------
>
> OOZIE-3513 Migrate from Preconditions.checkNotNull and ParamChecker.notNull
>
>
> Diffs
> -----
>
> client/src/main/java/org/apache/oozie/client/ApiJarFactory.java bb85d6780
> client/src/main/java/org/apache/oozie/client/ApiJarLoader.java 6ecf485ed
> core/src/main/java/org/apache/oozie/BaseLocalOozieClient.java 376ea1165
> core/src/main/java/org/apache/oozie/XException.java fb1a381bb
> core/src/main/java/org/apache/oozie/action/ActionExecutorException.java 9bc0e81b9
> core/src/main/java/org/apache/oozie/action/hadoop/GitActionExecutor.java fc4d65b83
> core/src/main/java/org/apache/oozie/action/hadoop/HadoopTokenHelper.java 1bdeb32f6
> core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java ec45fe495
> core/src/main/java/org/apache/oozie/action/hadoop/MapReduceActionExecutor.java a509e4d98
> core/src/main/java/org/apache/oozie/action/hadoop/ShareLibExcluder.java 9c45e5392
> core/src/main/java/org/apache/oozie/command/TransitionXCommand.java 262a78f5b
> core/src/main/java/org/apache/oozie/command/bundle/BundleJobResumeXCommand.java bc58be6ef
> core/src/main/java/org/apache/oozie/command/bundle/BundleSubmitXCommand.java ab88cac6c
> core/src/main/java/org/apache/oozie/command/coord/CoordActionNotificationXCommand.java d51f0d780
> core/src/main/java/org/apache/oozie/command/coord/CoordActionSkipXCommand.java ef0fc2d3e
> core/src/main/java/org/apache/oozie/command/coord/CoordActionTimeOutXCommand.java 9646d738a
> core/src/main/java/org/apache/oozie/command/coord/CoordActionsIgnoreXCommand.java 88842f1f1
> core/src/main/java/org/apache/oozie/command/coord/CoordRerunXCommand.java 2f158cdbf
> core/src/main/java/org/apache/oozie/command/coord/CoordSubmitXCommand.java 2622f05d9
> core/src/main/java/org/apache/oozie/command/wf/ReRunXCommand.java dd8d3d2f6
> core/src/main/java/org/apache/oozie/command/wf/SubmitHttpXCommand.java 9ee84063a
> core/src/main/java/org/apache/oozie/command/wf/SubmitXCommand.java 70b9adc1c
> core/src/main/java/org/apache/oozie/command/wf/WorkflowNotificationXCommand.java 62bf9b5b3
> core/src/main/java/org/apache/oozie/coord/CoordELFunctions.java 2761e44b0
> core/src/main/java/org/apache/oozie/coord/CoordUtils.java 1d97accd4
> core/src/main/java/org/apache/oozie/executor/jpa/BulkJPAExecutor.java 42961157a
> core/src/main/java/org/apache/oozie/executor/jpa/BundleActionGetJPAExecutor.java 3293efd69
> core/src/main/java/org/apache/oozie/executor/jpa/BundleActionInsertJPAExecutor.java 4b99c1a59
> core/src/main/java/org/apache/oozie/executor/jpa/BundleActionsCountForJobGetJPAExecutor.java c9f3a212d
> core/src/main/java/org/apache/oozie/executor/jpa/BundleJobGetCoordinatorsJPAExecutor.java 2f4665e15
> core/src/main/java/org/apache/oozie/executor/jpa/BundleJobGetForUserJPAExecutor.java aef9eb79c
> core/src/main/java/org/apache/oozie/executor/jpa/BundleJobGetJPAExecutor.java 7f98cd2a8
> core/src/main/java/org/apache/oozie/executor/jpa/BundleJobInfoGetJPAExecutor.java 6e1ea3e5b
> core/src/main/java/org/apache/oozie/executor/jpa/BundleJobInsertJPAExecutor.java e7373d67d
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionGetForCheckJPAExecutor.java 9e8b33191
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionGetForExternalIdJPAExecutor.java 61614e9fa
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionGetForInfoJPAExecutor.java 211d594e5
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionGetForInputCheckJPAExecutor.java 484ff93c9
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionGetForStartJPAExecutor.java 14b831d65
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionGetForTimeoutJPAExecutor.java c720be99d
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionGetJPAExecutor.java 53214ea64
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionInsertJPAExecutor.java 09e885793
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionRemoveJPAExecutor.java 8ea31a31d
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionsActiveCountJPAExecutor.java 384277a65
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionsCountForJobIdJPAExecutor.java f49848f40
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionsPendingFalseCountGetJPAExecutor.java e427835fd
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionsPendingFalseStatusCountGetJPAExecutor.java b32ea6e9e
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionByActionNumberJPAExecutor.java 49eb15886
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionForNominalTimeJPAExecutor.java 9a447b03f
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionModifiedDateForRangeJPAExecutor.java 2177514e0
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionRunningCountForRangeJPAExecutor.java 34223bb6e
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsJPAExecutor.java 4fb7abbad
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsNotCompletedJPAExecutor.java af0c0a3a7
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsRunningJPAExecutor.java b00193128
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsStatusJPAExecutor.java 073461f7b
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java 9db9daadc
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSuspendedJPAExecutor.java c70f0d264
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetJPAExecutor.java 2c21057af
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetReadyActionsJPAExecutor.java 3b1cda29c
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetRunningActionsCountJPAExecutor.java 567df40fa
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobInfoGetJPAExecutor.java 4c641d7e8
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobInsertJPAExecutor.java c26bc693d
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobsToBeMaterializedJPAExecutor.java 6c78bd126
> core/src/main/java/org/apache/oozie/executor/jpa/CoordinatorJobGetForUserAppnameJPAExecutor.java 42505eaee
> core/src/main/java/org/apache/oozie/executor/jpa/CoordinatorJobGetForUserJPAExecutor.java 58f5393a8
> core/src/main/java/org/apache/oozie/executor/jpa/SLAEventInsertJPAExecutor.java 8955419d2
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionGetJPAExecutor.java 1c7a14b0e
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionInsertJPAExecutor.java 637f2e083
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionRetryManualGetJPAExecutor.java 25ad7e232
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionSubsetGetJPAExecutor.java 9a8e10321
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionsGetForJobJPAExecutor.java 2f012a46e
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowIdGetForExternalIdJPAExecutor.java b0c92591c
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowInfoWithActionsSubsetGetJPAExecutor.java 5df684169
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowJobGetActionsJPAExecutor.java cd8586ee4
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowJobGetForUserJPAExecutor.java 7233bcae7
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowJobGetJPAExecutor.java 54350f461
> core/src/main/java/org/apache/oozie/executor/jpa/WorkflowJobInsertJPAExecutor.java 0816c27de
> core/src/main/java/org/apache/oozie/servlet/JsonRestServlet.java aca79baab
> core/src/main/java/org/apache/oozie/store/WorkflowStore.java a6b38c59e
> core/src/main/java/org/apache/oozie/util/ConfigUtils.java 895370a2b
> core/src/main/java/org/apache/oozie/util/FixedJsonInstanceSerializer.java f319fb9a5
> core/src/main/java/org/apache/oozie/util/IOUtils.java 1ee237b16
> core/src/main/java/org/apache/oozie/util/JobUtils.java 721a8a0e9
> core/src/main/java/org/apache/oozie/util/ParamChecker.java e86971f5d
> core/src/main/java/org/apache/oozie/util/ParameterVerifier.java ed323a846
> core/src/main/java/org/apache/oozie/util/PriorityDelayQueue.java 365f918dd
> core/src/main/java/org/apache/oozie/util/PropertiesUtils.java ec0f68725
> core/src/main/java/org/apache/oozie/util/XmlUtils.java 41a828127
> core/src/main/java/org/apache/oozie/util/db/OperationRetryHandler.java 53f2c3a6c
> core/src/main/java/org/apache/oozie/workflow/lite/ActionNodeDef.java 1e4852d17
> core/src/main/java/org/apache/oozie/workflow/lite/DBLiteWorkflowLib.java fbc1aef50
> core/src/main/java/org/apache/oozie/workflow/lite/DecisionNodeDef.java 13a3d8ada
> core/src/main/java/org/apache/oozie/workflow/lite/LiteWorkflowApp.java 5e6dc7a78
> core/src/main/java/org/apache/oozie/workflow/lite/LiteWorkflowInstance.java 560353c20
> core/src/main/java/org/apache/oozie/workflow/lite/LiteWorkflowLib.java 2e09889dc
> core/src/main/java/org/apache/oozie/workflow/lite/NodeDef.java b24b84488
> core/src/test/java/org/apache/oozie/test/XTestCase.java ed02ff6a3
> core/src/test/java/org/apache/oozie/util/TestParamChecker.java 76f2a9c99
> core/src/test/java/org/apache/oozie/util/TestParameterVerifier.java 15bd99a16
> core/src/test/java/org/apache/oozie/util/TestPriorityDelayQueue.java b48e0cc09
> examples/src/test/java/org/apache/oozie/example/TestLocalOozieExample.java 9b7bfe41e
> fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/mapping/DecisionConverter.java 7523436a0
> fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/mapping/ExplicitNodeConverter.java a890def2b
> fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/mapping/GlobalConverter.java dae2655a0
> fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/mapping/GraphNodesToWORKFLOWAPPConverter.java 00f4c4f2e
> fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/mapping/GraphToWORKFLOWAPPConverter.java b7d99dfc2
> server/src/main/java/org/apache/oozie/server/EmbeddedOozieServer.java e2bb7303e
> server/src/main/java/org/apache/oozie/server/FilterMapper.java b5d59fd11
> server/src/main/java/org/apache/oozie/server/HttpConfigurationWrapper.java 0341f9cea
> server/src/main/java/org/apache/oozie/server/JspHandler.java 6a6fbfa52
> server/src/main/java/org/apache/oozie/server/SSLServerConnectorFactory.java aea02a058
> server/src/main/java/org/apache/oozie/server/ServletMapper.java e0a135856
> sharelib/git/src/main/java/org/apache/oozie/action/hadoop/GitMain.java 692195c26
> sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/HdfsOperations.java 751d0e0f7
> sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherAM.java 801a0ae26
> sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherMain.java bc5f695f2
> tools/src/main/java/org/apache/oozie/tools/OozieDBImportCLI.java f91a453c9
> tools/src/main/java/org/apache/oozie/tools/OozieSharelibCLI.java 6c1195c2f
> tools/src/main/java/org/apache/oozie/tools/diag/ArgParser.java 115d0ba34
>
>
> Diff: https://reviews.apache.org/r/70964/diff/2/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Andras Salamon
>
>