NihalJain commented on code in PR #6131:
URL: https://github.com/apache/hbase/pull/6131#discussion_r1697928408


##########
hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/RestoreDriver.java:
##########
@@ -124,8 +124,8 @@ private int parseAndRun() throws IOException {
 
     if (cmd.hasOption(OPTION_YARN_QUEUE_NAME)) {
       String queueName = cmd.getOptionValue(OPTION_YARN_QUEUE_NAME);
-      // Set system property value for MR job
-      System.setProperty("mapreduce.job.queuename", queueName);
+      // Set MR job queuename to configuration
+      getConf().set("mapreduce.job.queuename", queueName);

Review Comment:
   Hi @2005hithlj  I was curious with one doubt? Does not simply passing 
-Dmapreduce.job.queuename=YOUR_QUEUE_NAME as args to the MapReduce job work?



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

Reply via email to