[ https://issues.apache.org/jira/browse/MAPREDUCE-7069?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16428382#comment-16428382 ]
Jim Brennan commented on MAPREDUCE-7069: ---------------------------------------- I put up a new patch that fixes the checkstyle/whitespace issues. I believe the unit test failure is unrelated. [~jlowe], I think this is ready for another review, unless you'd prefer to wait for the latest patch to finish automated testing. For setVMEnv(), I was able to rearrange the code to avoid calling MRApps.setEnvFromInputProperty twice. > Add ability to specify user environment variables individually > -------------------------------------------------------------- > > Key: MAPREDUCE-7069 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-7069 > Project: Hadoop Map/Reduce > Issue Type: Improvement > Reporter: Jim Brennan > Assignee: Jim Brennan > Priority: Major > Attachments: MAPREDUCE-7069.001.patch, MAPREDUCE-7069.002.patch, > MAPREDUCE-7069.003.patch, MAPREDUCE-7069.004.patch > > > As reported in YARN-6830, it is currently not possible to specify an > environment variable that contains commas via {{mapreduce.map.env}}, > mapreduce.reduce.env, or {{mapreduce.admin.user.env}}. > To address this, [~aw] proposed in [YARN-6830] that we add the ability to > specify environment variables individually: > {quote}e.g, mapreduce.map.env.[foo]=bar gets turned into foo=bar > {quote} -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org