sankarh commented on a change in pull request #642: HIVE-21776 : Replication 
fails to replicate a UDF with jar on HDFS during incremental
URL: https://github.com/apache/hive/pull/642#discussion_r287365528
 
 

 ##########
 File path: 
standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/ReplChangeManager.java
 ##########
 @@ -356,6 +356,9 @@ public static FileInfo getFileInfo(Path src, String 
checksumString, String srcCM
   // Currently using fileuri#checksum#cmrooturi#subdirs as the format
   public static String encodeFileUri(String fileUriStr, String fileChecksum, 
String encodedSubDir)
           throws IOException {
+    if (instance == null) {
+      throw new IOException("Uninitialized ReplChangeManager instance.");
+    }
     String encodedUri = fileUriStr;
 
 Review comment:
   I agree that we need not fix it right away as we currently enforce to enable 
CM for replication to work.
   But, in HDP 2.6, we already support replication even when CM is disabled 
which is broken in HDP 3.1. This is due to difference in implementation of 
isCMFileUri method in both these versions.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

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

Reply via email to