ayushtkn commented on a change in pull request #2043:
URL: https://github.com/apache/hive/pull/2043#discussion_r600581084



##########
File path: common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
##########
@@ -667,6 +667,17 @@ private static void populateLlapDaemonVarsSet(Set<String> 
llapDaemonVarsSetLocal
         + " table or partition level. If hive.exec.parallel \n"
         + "is set to true then max worker threads created for copy can be 
hive.exec.parallel.thread.number(determines \n"
         + "number of copy tasks in parallel) * hive.repl.parallel.copy.tasks 
"),
+    
REPL_SNAPSHOT_DIFF_FOR_EXTERNAL_TABLE_COPY("hive.repl.externaltable.snapshotdiff.copy",
+        false,"Use snapshot diff for copying data from source to "
+        + "destination cluster for external table in distcp"),
+    
REPL_SNAPSHOT_PREFIX_FOR_EXTERNAL_TABLE_COPY("hive.repl.externaltable.snapshot.prefix",

Review comment:
       Added validation in the code, Extended a test as well to confirm the 
validation. Making it user defined to prevent collision with any other distCp 
job with similar names.




-- 
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]



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

Reply via email to