-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70635/
-----------------------------------------------------------

(Updated May 20, 2019, 9:17 a.m.)


Review request for oozie.


Changes
-------

Rebased change


Bugs: OOZIE-3488
    https://issues.apache.org/jira/browse/OOZIE-3488


Repository: oozie-git


Description
-------

In Oozie, guava classes are used even when Java 8 already provides a compatible 
implementation, removing these usages would reduce the dependency to guava 
(which is regularly a headache due to her backward incompatible changes).
 This patch removes:
 * com.google.common.base.Function
 * com.google.common.collect.Iterables
 * com.google.common.collect.Ordering
 * com.google.common.collect.Lists
 * com.google.common.base.Predicate
 * com.google.common.base.Objects
 * com.google.common.base.Optional


Diffs (updated)
-----

  client/src/main/java/org/apache/oozie/client/OozieClient.java 54e45c0b 
  
core/src/main/java/org/apache/oozie/action/hadoop/MapReduceActionExecutor.java 
7ebc909a 
  core/src/main/java/org/apache/oozie/action/hadoop/SharelibResolver.java 
18c50cbf 
  core/src/main/java/org/apache/oozie/coord/CoordELFunctions.java e38f3d60 
  core/src/main/java/org/apache/oozie/service/JvmPauseMonitorService.java 
13b28098 
  core/src/main/java/org/apache/oozie/servlet/V1JobsServlet.java 61db5def 
  core/src/main/java/org/apache/oozie/util/db/CompositeIndex.java 88b9ca37 
  core/src/main/java/org/apache/oozie/util/db/DatabaseRetryPredicate.java 
c89aabe1 
  core/src/main/java/org/apache/oozie/util/db/FailingConnectionWrapper.java 
9e603ed0 
  core/src/main/java/org/apache/oozie/util/db/FailingDBHelperForTest.java 
7131e921 
  core/src/main/java/org/apache/oozie/util/db/FailingHSQLDBDriverWrapper.java 
5f5d56f6 
  core/src/main/java/org/apache/oozie/util/db/OperationRetryHandler.java 
16a0a827 
  
core/src/main/java/org/apache/oozie/util/db/PersistenceExceptionSubclassFilterRetryPredicate.java
 1e4413b6 
  core/src/main/java/org/apache/oozie/workflow/lite/LiteWorkflowValidator.java 
c7f2f498 
  
core/src/test/java/org/apache/oozie/action/hadoop/TestYarnApplicationIdFinder.java
 3fd7149f 
  core/src/test/java/org/apache/oozie/service/TestCallableQueueService.java 
aec1765b 
  core/src/test/java/org/apache/oozie/service/TestShareLibService.java 8bf2040c 
  core/src/test/java/org/apache/oozie/sla/TestSLACalculatorMemory.java e1960332 
  
core/src/test/java/org/apache/oozie/util/db/TestOozieDmlStatementPredicate.java 
c2d4b157 
  core/src/test/java/org/apache/oozie/util/db/TestOperationRetryHandler.java 
c61b1e55 
  
core/src/test/java/org/apache/oozie/util/db/TestPersistenceExceptionSubclassFilterRetryPredicate.java
 26dc13ba 
  
fluent-job/fluent-job-api/src/test/java/org/apache/oozie/fluentjob/api/mapping/SourceDataFactory.java
 4def062d 
  
fluent-job/fluent-job-api/src/test/java/org/apache/oozie/fluentjob/api/workflow/TestCredentialsBuilder.java
 4d3dce33 
  
fluent-job/fluent-job-api/src/test/java/org/apache/oozie/fluentjob/api/workflow/TestWorkflowBuilder.java
 29caf275 
  sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherAM.java 
6df151cc 
  
sharelib/oozie/src/test/java/org/apache/oozie/action/hadoop/TestLauncherAMUtils.java
 43c49144 
  
sharelib/spark/src/test/java/org/apache/oozie/action/hadoop/TestSparkArgsExtractor.java
 e86a2994 
  tools/src/main/java/org/apache/oozie/tools/OozieDBImportCLI.java 3db590d0 


Diff: https://reviews.apache.org/r/70635/diff/3/

Changes: https://reviews.apache.org/r/70635/diff/2-3/


Testing
-------

Tested locally


Thanks,

Zsombor Gegesy

Reply via email to