InvisibleProgrammer commented on code in PR #3798:
URL: https://github.com/apache/hive/pull/3798#discussion_r1288207432


##########
ql/src/java/org/apache/hadoop/hive/ql/exec/repl/AtlasDumpTask.java:
##########
@@ -129,6 +137,14 @@ public int execute() {
     }
   }
 
+  @NotNull
+  private void initializeReplLogger(AtlasReplInfo atlasReplInfo) {
+    if (this.replLogger == null){
+      this.replLogger = new AtlasDumpLogger(atlasReplInfo.getSrcDB(),
+              atlasReplInfo.getStagingDir().toString());
+    }
+  }

Review Comment:
   Ok, it is a tricky one: 
   
   I've tried to upgrade mockito. And the never version contains clean up: they 
removed obsolete methods that we currently use. And not only us, but powermock 
uses them as well. After upgrading mockito to 4.11.0, powermock tests stop 
running as they cannot find powermock methods. 
   And also, the older version of mockite-core is still in hive-exec as a 
transitive dependency from the root pom.xml and it conflicts with the 
mockito-inline dependency defined in hive-exec.
   
   My best idea to solve it is to remove mockito from the root pom.xml and I 
can move forward with hive-exec. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to