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

Varun Saxena commented on MAPREDUCE-6838:
-----------------------------------------

bq. Why are changing hadoop-common code to ignore ignoreUseIpConfig? Is this 
because to add test case? Does test is failing?
As discussed offline, this is to ensure that token service resolution is in 
sync between NM and AM. Also, with default configurations, E2E flow won't work.

bq. I think it is better to update token on each heart beat rather then 
comparing each time. It avoids proto compare.
This would involve iterating over all the tokens on each HB when we add token 
to UGI. Infact depending on how many tokens a AM has in UGI, this may be even 
more expensive.

bq. This appears mixing collectorManager i.e NMCollectorService authentication 
and MR side. 
The latest patch file contains code from YARN-7006. I used a dummy patch just 
to ensure QA runs. Will delete that patch as YARN-7006 has gone in.

bq. One thing I observe is TimelineClient is published using login user which 
is doAS. I think we should publish is as owner of application otherwise we end 
up in writing data into sub app application table as well.
Fair point. We can raise a separate JIRA for this. And create a proxy user 
based on app user to ensure timeline client publishes with that.

bq. I just noticed that TimelineDelegationTokenIdentifier#Renewer has methods 
renew/cancel. These creates V1 client. Does it going to be a problem?
This is not used by us for V2 anywhere and the Renewer class is annotated as 
Private. I guess no client would explicitly call it and even if they do, it 
should fail.

> [ATSv2 Security] Add timeline delegation token received in allocate response 
> to UGI
> -----------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-6838
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6838
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>            Reporter: Varun Saxena
>            Assignee: Varun Saxena
>              Labels: yarn-5355-merge-blocker
>             Fix For: YARN-5355
>
>         Attachments: MAPREDUCE-6838-YARN-5355.01.patch, 
> MAPREDUCE-6838-YARN-5355.02.patch, MAPREDUCE-6838-YARN-5355.03.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

Reply via email to