[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16364094#comment-16364094 ] Arun Suresh commented on YARN-7838: --- Ah - thought is wasn't included. Cool, i've updated the fix version to reflect the same. > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Critical > Labels: constraints > Fix For: 3.1.0 > > Attachments: YARN-7838.001.patch, YARN-7838.002.patch, > YARN-7838.003.patch, YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16364087#comment-16364087 ] Weiwei Yang commented on YARN-7838: --- Hi [~asuresh], [~leftnoteasy], this was already committed to branch-3.1. So it should be included in 3.1.0 already right? > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Critical > Labels: constraints > Fix For: 3.2.0 > > Attachments: YARN-7838.001.patch, YARN-7838.002.patch, > YARN-7838.003.patch, YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16364085#comment-16364085 ] Wangda Tan commented on YARN-7838: -- Since this is a self-contained code, and YARN-7822 is part of 3.1.0, I'm fine with backporting put this to branch-3.1. > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Critical > Labels: constraints > Fix For: 3.2.0 > > Attachments: YARN-7838.001.patch, YARN-7838.002.patch, > YARN-7838.003.patch, YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16364039#comment-16364039 ] Arun Suresh commented on YARN-7838: --- [~leftnoteasy], can we consider this for 3.1.0 ? Don't think this is really a disruptive change, and it is a useful inclusion, especially for demo purposes. Also note the constraint checker changes, implemented in YARN-7822, will anyway be available in 3.1.0 and will not be testable/demo-able until 3.1.x > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Critical > Labels: constraints > Fix For: 3.2.0 > > Attachments: YARN-7838.001.patch, YARN-7838.002.patch, > YARN-7838.003.patch, YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16359832#comment-16359832 ] Weiwei Yang commented on YARN-7838: --- Confirmed with [~leftnoteasy], branch was correct. Resolving this issue and will set fixed version later once target version 3.2.0 is available. > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Critical > Attachments: YARN-7838.001.patch, YARN-7838.002.patch, > YARN-7838.003.patch, YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16359819#comment-16359819 ] Hudson commented on YARN-7838: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #13642 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/13642/]) YARN-7838. Support AND/OR constraints in Distributed Shell. Contributed (wwei: rev a08c048832d68c203fbdfce8d9f0e7dcccb02a55) * (add) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/util/constraint/package-info.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-applications-distributedshell/src/main/java/org/apache/hadoop/yarn/applications/distributedshell/ApplicationMaster.java * (add) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/test/java/org/apache/hadoop/yarn/api/resource/TestPlacementConstraintParser.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-applications-distributedshell/src/main/java/org/apache/hadoop/yarn/applications/distributedshell/PlacementSpec.java * (add) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/util/constraint/PlacementConstraintParseException.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-applications-distributedshell/src/main/java/org/apache/hadoop/yarn/applications/distributedshell/Client.java * (add) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/util/constraint/PlacementConstraintParser.java > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Major > Attachments: YARN-7838.001.patch, YARN-7838.002.patch, > YARN-7838.003.patch, YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16359816#comment-16359816 ] Weiwei Yang commented on YARN-7838: --- I think misunderstood [~leftnoteasy]'s email about 3.1.0 cut message... I think this should not be committed to branch-3.1. Double checking with [~leftnoteasy] and [~asuresh] to make sure this is committed to the correct branch. > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Major > Attachments: YARN-7838.001.patch, YARN-7838.002.patch, > YARN-7838.003.patch, YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16359804#comment-16359804 ] Weiwei Yang commented on YARN-7838: --- Thanks [~asuresh], I will commit this soon to trunk and branch 3.1. Since 3.1.0 was just cut and code frozen, this will not go in 3.1.0. > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Major > Attachments: YARN-7838.001.patch, YARN-7838.002.patch, > YARN-7838.003.patch, YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16359779#comment-16359779 ] Arun Suresh commented on YARN-7838: --- Thanks for the patch [~cheersyang]. It looks fine to me. +1 Do commit this to branch 3.1 as well.. > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Major > Attachments: YARN-7838.001.patch, YARN-7838.002.patch, > YARN-7838.003.patch, YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16354915#comment-16354915 ] genericqa commented on YARN-7838: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 25s{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:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 48s{color} | {color:blue} Maven dependency ordering for branch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 15m 35s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 9m 35s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 54s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 2s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 11m 8s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 1m 10s{color} | {color:red} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api in trunk has 1 extant Findbugs warnings. {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 43s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 10s{color} | {color:blue} Maven dependency ordering for patch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 48s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 8m 36s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 8m 36s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 55s{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} 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} 10m 3s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 2m 4s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 40s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 41s{color} | {color:green} hadoop-yarn-api in the patch passed. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 13m 16s{color} | {color:green} hadoop-yarn-applications-distributedshell in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 30s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 80m 11s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:5b98639 | | JIRA Issue | YARN-7838 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12909541/YARN-7838.003.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 78dfad9fe10e 4.4.0-89-generic #112-Ubuntu SMP Mon Jul 31 19:38:41 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 4fadcf1 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_151 | | findbugs | v3.1.0-RC1 | | findbugs | https://builds.apache.org/job/PreCommit-YARN-Build/19631/a
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16353925#comment-16353925 ] genericqa commented on YARN-7838: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 10m 56s{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:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 1m 6s{color} | {color:blue} Maven dependency ordering for branch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 17m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 8m 28s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 1m 1s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 7s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 11m 32s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 1m 18s{color} | {color:red} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api in trunk has 1 extant Findbugs warnings. {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 41s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 10s{color} | {color:blue} Maven dependency ordering for patch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 54s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 7m 3s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 7m 2s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 55s{color} | {color:orange} hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 141 unchanged - 0 fixed = 142 total (was 141) {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} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:red}-1{color} | {color:red} shadedclient {color} | {color:red} 9m 45s{color} | {color:red} patch has errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 2m 17s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} javadoc {color} | {color:red} 0m 26s{color} | {color:red} hadoop-yarn-project_hadoop-yarn_hadoop-yarn-api generated 1 new + 1 unchanged - 0 fixed = 2 total (was 1) {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 43s{color} | {color:green} hadoop-yarn-api in the patch passed. {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 12m 46s{color} | {color:red} hadoop-yarn-applications-distributedshell in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 34s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 90m 41s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.yarn.applications.distributedshell.TestDistributedShell | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:5b98639 | | JIRA Issue | YARN-7838 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12909434/YARN-7838.002.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux d9110a5f6b6e 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 | | P
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16353819#comment-16353819 ] Weiwei Yang commented on YARN-7838: --- Fixed jenkins issues... > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Major > Attachments: YARN-7838.001.patch, YARN-7838.002.patch, > YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16353362#comment-16353362 ] genericqa commented on YARN-7838: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 29s{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:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 47s{color} | {color:blue} Maven dependency ordering for branch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 15m 2s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 7m 23s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 58s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 7s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 11m 35s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 1m 5s{color} | {color:red} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api in trunk has 1 extant Findbugs warnings. {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 53s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 10s{color} | {color:blue} Maven dependency ordering for patch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 49s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 6m 32s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 6m 32s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 0m 55s{color} | {color:orange} hadoop-yarn-project/hadoop-yarn: The patch generated 7 new + 141 unchanged - 0 fixed = 148 total (was 141) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 1s{color} | {color:green} the patch passed {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} 10m 5s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 0m 41s{color} | {color:red} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-applications-distributedshell generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0) {color} | | {color:red}-1{color} | {color:red} javadoc {color} | {color:red} 0m 23s{color} | {color:red} hadoop-yarn-project_hadoop-yarn_hadoop-yarn-api generated 1 new + 1 unchanged - 0 fixed = 2 total (was 1) {color} | || || || || {color:brown} Other Tests {color} || | {color:red}-1{color} | {color:red} unit {color} | {color:red} 0m 39s{color} | {color:red} hadoop-yarn-api in the patch failed. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 12m 37s{color} | {color:green} hadoop-yarn-applications-distributedshell in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 30s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 75m 29s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | FindBugs | module:hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-applications-distributedshell | | | Found reliance on default encoding in org.apache.hadoop.yarn.applications.distributedshell.ApplicationMaster.parsePlacementSpecs(String):in org.apache.hadoop.yarn.applications.distributedshell.ApplicationMaster.parsePlacementSpecs(String): new String(byte[]) At ApplicationMaster.java:[line 677] | | | Found reliance on de
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16353324#comment-16353324 ] Weiwei Yang commented on YARN-7838: --- Hi [~asuresh] I just submitted a formal v1 patch, this patch adds a {{PlacementConstraintParser}}, and {{PlacementSpec}} now simply calls {{parsePlacementSpec}} to parse a string expression. Syntax is just like before {code} Tag1=N1,P1:Tag2=N2,P2:...:TagN=Nn,Pn {code} where each KV pair for tag represents source tag and number of allocations, such as {{foo=3}}. where each {{Pn}} can be any form of a valid constraint expression (include nested forms), such as {code} in,node,foo,bar notin,node,foo,bar,1,2 and(notin,node,foo:notin,node,bar) and(notin,node,foo:or(in,node.bar:in,node,moo)) {code} a comprehensive UT can be found in {{TestPlacementConstraintParser}}. I have also tried end-to-end testing with different combination of constraints, includes {code} -placement_spec foo=4,notin,node,foo -placement_spec foo=3,NOTIN,NODE,foo:bar=1,IN,NODE,foo -placement_spec "foo=4,AND(notin,node,foo:cardinality,node,foo,0,2)" {code} all seem to work fine to me. Please check and let me know your feedback. Thanks > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Major > Attachments: YARN-7838.001.patch, YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16350798#comment-16350798 ] Arun Suresh commented on YARN-7838: --- Thanks for taking a stab at this [~cheersyang]. Yup, I agree we need a more flexible parser - the placementspec parser I put in was just for some adhoc testing :) Couple of comments: # do we need a tryParse ? Either we are able to parse or an exception is thrown right ? # The {{toInt}} should be static # I am assuming your {{shouldHaveNext}} is more like an assert - Maybe make that static as well, and # In the final implementation, we have to ensure that it accepts a placementspec string WITHOUT any and/or as well. > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Major > Attachments: YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-7838) Support AND/OR constraints in Distributed Shell
[ https://issues.apache.org/jira/browse/YARN-7838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16350372#comment-16350372 ] Weiwei Yang commented on YARN-7838: --- Hello [~asuresh] Today I spent a few hours working on this one, to be able to support composite and nested constraints in DS, I think today's approach in PlacementSpec is not flexible. So I created a parser class {{PlacementConstraintParser}}. This is a prelim patch, please take a look and let me know your feedback. My thought is we can use such parser class to further support specifying expressions while submit an app, similar format like in DS. So app could be easier to use this feature without modifying client code. Once you agree with this approach, I can go on working on a formal patch. Thank you. > Support AND/OR constraints in Distributed Shell > --- > > Key: YARN-7838 > URL: https://issues.apache.org/jira/browse/YARN-7838 > Project: Hadoop YARN > Issue Type: Sub-task > Components: distributed-shell >Reporter: Weiwei Yang >Assignee: Weiwei Yang >Priority: Major > Attachments: YARN-7838.prelim.patch > > > Extending DS placement spec syntax to support AND/OR constraints, something > like > {code} > // simple > -placement_spec foo=4,AND(NOTIN,NODE,foo:NOTIN,NODE,bar) > // nested > -placement_spec foo=4,AND(NOTIN,NODE,foo:OR(IN,NODE,moo:IN,NODE,bar)) > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org