[ 
https://issues.apache.org/jira/browse/TEZ-3627?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15872505#comment-15872505
 ] 

Siddharth Seth commented on TEZ-3627:
-------------------------------------

Minor: queueName in the context should be under a lock / volatile - for 
visibility. The scheduler thread is different from the thread that will read 
the value.
Renaming it to queue name makes sense to me.

Other than that, looks good. Thanks!

> Use queue name available in RegisterApplicationMasterResponse.
> --------------------------------------------------------------
>
>                 Key: TEZ-3627
>                 URL: https://issues.apache.org/jira/browse/TEZ-3627
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Harish Jaiprakash
>            Assignee: Harish Jaiprakash
>         Attachments: TEZ-3627.01.patch, TEZ-3627.02.patch
>
>
> The queue name was already being sent in RegisterApplicationMasterResponse 
> from yarn. Use that instead of the environment variable to set the queue name.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to