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

Rajesh Balamohan commented on TEZ-4254:
---------------------------------------

Had offline conversation with [~mustafaiman] . If config has something like 
"tez.runtime.shuffle.ssl.enable=${mapreduce.shuffle.ssl.enabled}" (ie via 
references), then it can lead to issue when we unset in config.

> Don't unset the tez config if both mr and tez config have same value.
> ---------------------------------------------------------------------
>
>                 Key: TEZ-4254
>                 URL: https://issues.apache.org/jira/browse/TEZ-4254
>             Project: Apache Tez
>          Issue Type: Bug
>    Affects Versions: 0.9.1
>            Reporter: Rajkumar Singh
>            Priority: Major
>         Attachments: TEZ-4254.patch
>
>
> while translated mr config to the tez config we unset config here.
> https://github.com/apache/tez/blob/master/tez-mapreduce/src/main/java/org/apache/tez/mapreduce/hadoop/MRHelpers.java#L187
> There is no need to unset the config if both mr and tez have same value.
> we observed if equivalent of tez config for mapreduce.shuffle.ssl.enabled 
> (tez.runtime.shuffle.ssl.enable) set to true, we unset the config, this 
> propagate config 
> tez.runtime.shuffle.ssl.enable={mapreduce.shuffle.ssl.enabled} at runtime and 
> variable substitution does not work correctly in the absence of mrconfig 
> runtime.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to