[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17803059#comment-17803059 ] Shilun Fan commented on HADOOP-14498: - Bulk update: moved all 3.4.0 non-blocker issues, please move back if it is a blocker. Retarget 3.5.0. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Assignee: Sean Mackrory >Priority: Major > Attachments: HADOOP-14498.001.patch, HADOOP-14498.002.patch, > HADOOP-14498.003.patch > > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17080898#comment-17080898 ] Hadoop QA commented on HADOOP-14498: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 34s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 27m 35s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 20s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 14m 7s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 52s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 0s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} shellcheck {color} | {color:green} 0m 0s{color} | {color:green} There were no new shellcheck issues. {color} | | {color:green}+1{color} | {color:green} shelldocs {color} | {color:green} 0m 12s{color} | {color:green} The patch generated 0 new + 104 unchanged - 132 fixed = 104 total (was 236) {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 13m 8s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 2m 53s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 33s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 63m 1s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=19.03.8 Server=19.03.8 Image:yetus/hadoop:e6455cc864d | | JIRA Issue | HADOOP-14498 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12880256/HADOOP-14498.003.patch | | Optional Tests | dupname asflicense mvnsite unit shellcheck shelldocs | | uname | Linux 8ef08cccf7b3 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 275c478 | | maven | version: Apache Maven 3.3.9 | | shellcheck | v0.3.7 | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/16865/testReport/ | | Max. process+thread count | 454 (vs. ulimit of 5500) | | modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/16865/console | | Powered by | Apache Yetus 0.8.0 http://yetus.apache.org | This message was automatically generated. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Assignee: Sean Mackrory >Priority: Major > Attachments: HADOOP-14498.001.patch, HADOOP-14498.002.patch, > HADOOP-14498.003.patch > > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16148479#comment-16148479 ] Allen Wittenauer commented on HADOOP-14498: --- I think this fixes the immediate problem, but I'm wondering if we shouldn't purge the usage of hadoop_add_entry and hadoop_verify_entry while we're here... especially since it is only used by the HOT code anyway. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Assignee: Sean Mackrory > Attachments: HADOOP-14498.001.patch, HADOOP-14498.002.patch, > HADOOP-14498.003.patch > > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16131520#comment-16131520 ] Andrew Wang commented on HADOOP-14498: -- Hey [~aw], do you have time to review Sean's patch? > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Assignee: Sean Mackrory >Priority: Critical > Attachments: HADOOP-14498.001.patch, HADOOP-14498.002.patch, > HADOOP-14498.003.patch > > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16113143#comment-16113143 ] Hadoop QA commented on HADOOP-14498: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 17s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 14m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 27s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 21s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} shellcheck {color} | {color:green} 0m 4s{color} | {color:green} There were no new shellcheck issues. {color} | | {color:green}+1{color} | {color:green} shelldocs {color} | {color:green} 0m 9s{color} | {color:green} There were no new shelldocs issues. {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 2m 6s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 17s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 20m 46s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:14b5c93 | | JIRA Issue | HADOOP-14498 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12880256/HADOOP-14498.003.patch | | Optional Tests | asflicense mvnsite unit shellcheck shelldocs | | uname | Linux b531bc262252 3.13.0-119-generic #166-Ubuntu SMP Wed May 3 12:18:55 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 293c74a | | shellcheck | v0.4.6 | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/12943/testReport/ | | modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/12943/console | | Powered by | Apache Yetus 0.6.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Assignee: Sean Mackrory >Priority: Critical > Attachments: HADOOP-14498.001.patch, HADOOP-14498.002.patch, > HADOOP-14498.003.patch > > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16112062#comment-16112062 ] Hadoop QA commented on HADOOP-14498: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 23s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 14m 2s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 23s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 15s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} shellcheck {color} | {color:red} 0m 4s{color} | {color:red} The patch generated 4 new + 20 unchanged - 0 fixed = 24 total (was 20) {color} | | {color:green}+1{color} | {color:green} shelldocs {color} | {color:green} 0m 9s{color} | {color:green} There were no new shelldocs issues. {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 1m 50s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 15s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 19m 46s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:14b5c93 | | JIRA Issue | HADOOP-14498 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12880140/HADOOP-14498.002.patch | | Optional Tests | asflicense mvnsite unit shellcheck shelldocs | | uname | Linux de9b9a42bfa8 4.4.0-43-generic #63-Ubuntu SMP Wed Oct 12 13:48:03 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 79df1e7 | | shellcheck | v0.4.6 | | shellcheck | https://builds.apache.org/job/PreCommit-HADOOP-Build/12934/artifact/patchprocess/diff-patch-shellcheck.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/12934/testReport/ | | modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/12934/console | | Powered by | Apache Yetus 0.6.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Assignee: Sean Mackrory >Priority: Critical > Attachments: HADOOP-14498.001.patch, HADOOP-14498.002.patch > > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16112029#comment-16112029 ] Allen Wittenauer commented on HADOOP-14498: --- bq. I guess that's what the comment means by "different syntaxes". Yeah. In the beginning I was trying really really really hard to avoid arrays, for lots of reasons. One of the big ones that I'm willing to write down was that not all of the array functions are available in our target bash 3.2. (e.g., associative arrays, mapfile, etc). Plus backward compatibility with the raw string format of HADOOP_OPTS + trying to solve the duplicate parameter problem led to add_param. It's not pretty and I'm not proud of it. But it works. Sidenote: HADOOP_OPTS is probably at this point the biggest hindsight-20-20 mistake in Hadoop. I don't think people really understand how much of an impact it's had on literally everything in the system. For example, it's *the* reason that spaces in file paths are a complete nightmare. HADOOP-13365 is my attempt at fixing it. I'm not sure if it makes it worse or better though. bq. It'd be nice to replace that inline loop with a join function. Yeah, I'd love for someone to take another whack at it. I can't remember what all I tried before I ended up just settling on the loop. I seem to recall I had a better way, but it only worked with bash 4.x. I guess we could always put a version check in there. (There's one or two other places like that already.) bq. Or should I just throw it in hadoop-functions.sh? I've just been throwing everything into hadoop-functions.sh, as it ends up creating one big API doc at mvn site time. Pretty convenient. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Assignee: Sean Mackrory >Priority: Critical > Attachments: HADOOP-14498.001.patch, HADOOP-14498.002.patch > > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16111994#comment-16111994 ] Sean Mackrory commented on HADOOP-14498: Aah that makes sense. I guess that's what the comment means by "different syntaxes". Testing a patch now that uses an array for HADOOP_SHELL_PROFILES and a corresponding test. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Assignee: Sean Mackrory >Priority: Critical > Attachments: HADOOP-14498.001.patch > > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16111959#comment-16111959 ] Allen Wittenauer commented on HADOOP-14498: --- The -00 patch will likely break the world. e.g.: {code} hadoop_add_param HADOOP_OPTS Xmx "-Xmx${HADOOP_HEAPSIZE_MAX}" {code} will now add multiple Xmx lines. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Assignee: Sean Mackrory >Priority: Critical > Attachments: HADOOP-14498.001.patch > > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16111952#comment-16111952 ] Allen Wittenauer commented on HADOOP-14498: --- The thing is, is that add_param was really meant for dealing with de-duping stuff like HADOOP_OPTS. arrays are probably a better choice here because we know we can do an exact match. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Assignee: Sean Mackrory >Priority: Critical > Attachments: HADOOP-14498.001.patch > > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16039705#comment-16039705 ] Mingliang Liu commented on HADOOP-14498: Thanks [~aw] for the prompt debugging. Your analysis makes sense to me. I'll have a look at [HADOOP-13595]. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Priority: Critical > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16039692#comment-16039692 ] Allen Wittenauer commented on HADOOP-14498: --- OK, using 'hadoop --debug classpath' we can see exactly what is happening with the specific construction of "hadoop-azure,hadoop-aws,hadoop-azure-datalake" {code} $ bin/hadoop --debug classpath 2>&1 | grep PROFILES DEBUG: HADOOP_SHELL_PROFILES accepted hadoop-aws DEBUG: HADOOP_SHELL_PROFILES accepted hadoop-azure-datalake DEBUG: HADOOP_SHELL_PROFILES declined hadoop-azure DEBUG: HADOOP_SHELL_PROFILES accepted hdfs DEBUG: HADOOP_SHELL_PROFILES accepted mapred DEBUG: HADOOP_SHELL_PROFILES accepted yarn {code} hadoop-azure is getting rejected by the shell profiles code because it is getting up in the dedupe pattern match code. Converting this to use the new array code added will probably fix this. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Priority: Critical > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14498) HADOOP_OPTIONAL_TOOLS not parsed correctly
[ https://issues.apache.org/jira/browse/HADOOP-14498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16039655#comment-16039655 ] Allen Wittenauer commented on HADOOP-14498: --- HADOOP\_OPTIONAL\_TOOLS basically triggers a read of "libexec/shellprofile.d/(whatever).sh" that is created at build time by some maven magic and "dev-support/bin/dist-tools-hooks-maker" . The inside of this file (after cutting out the boiler plate) says, effectively: {code} function _hadoop-azure-datalake_hadoop_classpath { if [[ -f "${HADOOP_TOOLS_HOME}/${HADOOP_TOOLS_LIB_JARS_DIR}/azure-data-lake-store-sdk-2.1.4.jar" ]]; then hadoop_add_classpath "${HADOOP_TOOLS_HOME}/${HADOOP_TOOLS_LIB_JARS_DIR}/azure-data-lake-store-sdk-2.1.4.jar" fi hadoop_add_classpath "${HADOOP_TOOLS_HOME}/${HADOOP_TOOLS_LIB_JARS_DIR}/hadoop-azure-datalake-3.0.0-alpha4-SNAPSHOT.jar" } {code} ie, we're going to add azure-data-lake-store-sdk-2.1.4.jar and hadoop-azure-datalake-3.0.0-alpha4-SNAPSHOT.jar to the classpath. hadoop-azure.sh, meanwhile, says: {code} if [[ -f "${HADOOP_TOOLS_HOME}/${HADOOP_TOOLS_LIB_JARS_DIR}/azure-storage-4.2.0.jar" ]]; then hadoop_add_classpath "${HADOOP_TOOLS_HOME}/${HADOOP_TOOLS_LIB_JARS_DIR}/azure-storage-4.2.0.jar" fi if [[ -f "${HADOOP_TOOLS_HOME}/${HADOOP_TOOLS_LIB_JARS_DIR}/azure-keyvault-core-0.8.0.jar" ]]; then hadoop_add_classpath "${HADOOP_TOOLS_HOME}/${HADOOP_TOOLS_LIB_JARS_DIR}/azure-keyvault-core-0.8.0.jar" fi hadoop_add_classpath "${HADOOP_TOOLS_HOME}/${HADOOP_TOOLS_LIB_JARS_DIR}/hadoop-azure-3.0.0-alpha4-SNAPSHOT.jar" {code} ie., azure-storage-4.2.0.jar, azure-keyvault-core-0.8.0.jar, and hadoop-azure-3.0.0-alpha4-SNAPSHOT.jar. Different dependencies are getting generated by the build and are either incorrect/incomplete in the pom, a bug in the dependency file generator, or something else going haywire. It is not a bug in how HADOOP\_OPTIONAL\_TOOLS is getting parsed post-build. > HADOOP_OPTIONAL_TOOLS not parsed correctly > -- > > Key: HADOOP-14498 > URL: https://issues.apache.org/jira/browse/HADOOP-14498 > Project: Hadoop Common > Issue Type: Bug > Components: build >Affects Versions: 3.0.0-alpha1 >Reporter: Mingliang Liu >Priority: Critical > > # This will make hadoop-azure not show up in the hadoop classpath, though > both hadoop-aws and hadoop-azure-datalake are in the > classpath.{code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws,hadoop-azure-datalake" > {code} > # And if we put only hadoop-azure and hadoop-aws, both of them are shown in > the classpath. > {code:title=hadoop-env.sh} > export HADOOP_OPTIONAL_TOOLS="hadoop-azure,hadoop-aws" > {code} > This makes me guess that, while parsing the {{HADOOP_OPTIONAL_TOOLS}}, we > make some assumptions that hadoop tool modules have a single "-" in names, > and the _hadoop-azure-datalake_ overrides the _hadoop-azure_. Or any other > assumptions about the {{${project.artifactId\}}}? > Ping [~aw]. -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org