Uploaded image for project: 'Hadoop YARN'
  1. Hadoop YARN
  2. YARN-3523

Cleanup ResourceManagerAdministrationProtocol interface audience

    Details

    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      I noticed ResourceManagerAdministrationProtocol has @Private audience for the class and @Public audience for methods. It doesn't make sense to me. We should make class audience and methods audience consistent.

      1. YARN-3523.20150422-1.patch
        1 kB
        Naganarasimha G R
      2. YARN-3523.20150504-1.patch
        3 kB
        Naganarasimha G R
      3. YARN-3523.20150505-1.patch
        4 kB
        Naganarasimha G R

        Activity

        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Hi Tan, Wangda,
        IIUC we need to make all the methods to have @Private audience right, as all methods over here are for administrative purpose ?
        if you already have a patch for this, please feel free to reassign

        Show
        Naganarasimha Naganarasimha G R added a comment - Hi Tan, Wangda , IIUC we need to make all the methods to have @Private audience right, as all methods over here are for administrative purpose ? if you already have a patch for this, please feel free to reassign
        Hide
        leftnoteasy Wangda Tan added a comment -

        Naganarasimha G R, thanks for taking this.
        Actually I'm not sure about what should be the correct audience setting for ResourceManagerAdministrationProtocol, as a public API, it should be @Public. But actually, it will only used by RMAdminCLI, I don't know is there any 3rd party projects writing their own "admin CLI" and implements ResourceManagerAdministrationProtocol.

        To not break compatibility, I think simple solution is to only change ResourceManagerAdministrationProtocol from @Private to @Public. I don't know if there's any thoughts about this.

        Show
        leftnoteasy Wangda Tan added a comment - Naganarasimha G R , thanks for taking this. Actually I'm not sure about what should be the correct audience setting for ResourceManagerAdministrationProtocol, as a public API, it should be @Public. But actually, it will only used by RMAdminCLI, I don't know is there any 3rd party projects writing their own "admin CLI" and implements ResourceManagerAdministrationProtocol. To not break compatibility, I think simple solution is to only change ResourceManagerAdministrationProtocol from @Private to @Public. I don't know if there's any thoughts about this.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        uploading with initial patch

        Show
        Naganarasimha Naganarasimha G R added a comment - uploading with initial patch
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 14m 31s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
        +1 whitespace 0m 0s The patch has no lines that end in whitespace.
        +1 javac 7m 32s There were no new javac warning messages.
        +1 javadoc 9m 34s There were no new javadoc warning messages.
        +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
        +1 checkstyle 3m 55s There were no new checkstyle issues.
        +1 install 1m 34s mvn install still works.
        +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
        +1 findbugs 1m 21s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
        +1 yarn tests 0m 23s Tests passed in hadoop-yarn-api.
            39m 48s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12727198/YARN-3523.20150422-1.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / b08908a
        hadoop-yarn-api test log https://builds.apache.org/job/PreCommit-YARN-Build/7445/artifact/patchprocess/testrun_hadoop-yarn-api.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/7445/testReport/
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/7445//console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 31s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 javac 7m 32s There were no new javac warning messages. +1 javadoc 9m 34s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 3m 55s There were no new checkstyle issues. +1 install 1m 34s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 1m 21s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 yarn tests 0m 23s Tests passed in hadoop-yarn-api.     39m 48s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12727198/YARN-3523.20150422-1.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / b08908a hadoop-yarn-api test log https://builds.apache.org/job/PreCommit-YARN-Build/7445/artifact/patchprocess/testrun_hadoop-yarn-api.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/7445/testReport/ Console output https://builds.apache.org/job/PreCommit-YARN-Build/7445//console This message was automatically generated.
        Hide
        gtCarrera9 Li Lu added a comment -

        Thanks Naganarasimha G R! The quick fix LGTM.

        Show
        gtCarrera9 Li Lu added a comment - Thanks Naganarasimha G R ! The quick fix LGTM.
        Hide
        vinodkv Vinod Kumar Vavilapalli added a comment -

        Naganarasimha G R, I've seen other tickets where you set fix-version. Our protocol is for contributors set target-version if needed and committers setting fix-version at commit-time. Fixing this one by myself, but please take that note for future.

        Show
        vinodkv Vinod Kumar Vavilapalli added a comment - Naganarasimha G R , I've seen other tickets where you set fix-version. Our protocol is for contributors set target-version if needed and committers setting fix-version at commit-time. Fixing this one by myself, but please take that note for future.
        Hide
        vinodkv Vinod Kumar Vavilapalli added a comment -

        Actually I'm not sure about what should be the correct audience setting for ResourceManagerAdministrationProtocol, as a public API, it should be @Public. But actually, it will only used by RMAdminCLI, I don't know is there any 3rd party projects writing their own "admin CLI" and implements IIRC,

        I originally intended ResourceManagerAdministrationProtocol to be private. But folks put Public annotation on the methods unfortunately.

        I just checked the api docs, it doesn't seem to appear there. Can others confirm too? If confirmed, we can simply mark everything as private.

        Show
        vinodkv Vinod Kumar Vavilapalli added a comment - Actually I'm not sure about what should be the correct audience setting for ResourceManagerAdministrationProtocol, as a public API, it should be @Public. But actually, it will only used by RMAdminCLI, I don't know is there any 3rd party projects writing their own "admin CLI" and implements IIRC, I originally intended ResourceManagerAdministrationProtocol to be private. But folks put Public annotation on the methods unfortunately. I just checked the api docs, it doesn't seem to appear there. Can others confirm too? If confirmed, we can simply mark everything as private.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Hi Vinod Kumar Vavilapalli, Point noted , as per my knowledge i think mistakenly set in 2 jiras and it was not intentional (if so would have set to earlier version than 2.8.0). My 2 cents here, if there is an option in the jira to disable/enable editing for particular group then can we apply that such that Fix version can be added/modified by committers only ?

        Show
        Naganarasimha Naganarasimha G R added a comment - Hi Vinod Kumar Vavilapalli , Point noted , as per my knowledge i think mistakenly set in 2 jiras and it was not intentional (if so would have set to earlier version than 2.8.0). My 2 cents here, if there is an option in the jira to disable/enable editing for particular group then can we apply that such that Fix version can be added/modified by committers only ?
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Have checked the 2.7.0 api docs and neither this class nor its package has been captured. Hence have modified visibility of the methods as private in this updated patch

        Show
        Naganarasimha Naganarasimha G R added a comment - Have checked the 2.7.0 api docs and neither this class nor its package has been captured. Hence have modified visibility of the methods as private in this updated patch
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 14m 55s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
        +1 javac 7m 42s There were no new javac warning messages.
        +1 javadoc 9m 52s There were no new javadoc warning messages.
        +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
        -1 checkstyle 1m 2s The applied patch generated 1 new checkstyle issues (total was 17, now 18).
        -1 whitespace 0m 0s The patch has 6 line(s) that end in whitespace. Use git apply --whitespace=fix.
        +1 install 1m 36s mvn install still works.
        +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse.
        +1 findbugs 1m 24s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
        +1 yarn tests 0m 27s Tests passed in hadoop-yarn-api.
            38m 5s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12730182/YARN-3523.20150504-1.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / bb9ddef
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/7686/artifact/patchprocess/diffcheckstylehadoop-yarn-api.txt
        whitespace https://builds.apache.org/job/PreCommit-YARN-Build/7686/artifact/patchprocess/whitespace.txt
        hadoop-yarn-api test log https://builds.apache.org/job/PreCommit-YARN-Build/7686/artifact/patchprocess/testrun_hadoop-yarn-api.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/7686/testReport/
        Java 1.7.0_55
        uname Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/7686/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 55s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac 7m 42s There were no new javac warning messages. +1 javadoc 9m 52s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 1m 2s The applied patch generated 1 new checkstyle issues (total was 17, now 18). -1 whitespace 0m 0s The patch has 6 line(s) that end in whitespace. Use git apply --whitespace=fix. +1 install 1m 36s mvn install still works. +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse. +1 findbugs 1m 24s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 yarn tests 0m 27s Tests passed in hadoop-yarn-api.     38m 5s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12730182/YARN-3523.20150504-1.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / bb9ddef checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/7686/artifact/patchprocess/diffcheckstylehadoop-yarn-api.txt whitespace https://builds.apache.org/job/PreCommit-YARN-Build/7686/artifact/patchprocess/whitespace.txt hadoop-yarn-api test log https://builds.apache.org/job/PreCommit-YARN-Build/7686/artifact/patchprocess/testrun_hadoop-yarn-api.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/7686/testReport/ Java 1.7.0_55 uname Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-YARN-Build/7686/console This message was automatically generated.
        Hide
        leftnoteasy Wangda Tan added a comment -

        I think we should also mark all interface to be @Unstable instead of @Stable. Thoughts, Vinod Kumar Vavilapalli?

        Show
        leftnoteasy Wangda Tan added a comment - I think we should also mark all interface to be @Unstable instead of @Stable. Thoughts, Vinod Kumar Vavilapalli ?
        Hide
        vinodkv Vinod Kumar Vavilapalli added a comment -

        Doesn't matter much for private APIs, but sure..

        Show
        vinodkv Vinod Kumar Vavilapalli added a comment - Doesn't matter much for private APIs, but sure..
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        in that case better to remove @Stable and not add @Unstable .. thoughts ?

        Show
        Naganarasimha Naganarasimha G R added a comment - in that case better to remove @Stable and not add @Unstable .. thoughts ?
        Hide
        vinodkv Vinod Kumar Vavilapalli added a comment -

        Makes sense.

        Show
        vinodkv Vinod Kumar Vavilapalli added a comment - Makes sense.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        updated the patch with removal of @stable

        Show
        Naganarasimha Naganarasimha G R added a comment - updated the patch with removal of @stable
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 15m 12s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
        +1 javac 7m 53s There were no new javac warning messages.
        +1 javadoc 10m 0s There were no new javadoc warning messages.
        +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
        +1 checkstyle 1m 6s There were no new checkstyle issues.
        +1 whitespace 0m 0s The patch has no lines that end in whitespace.
        +1 install 1m 35s mvn install still works.
        +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse.
        +1 findbugs 1m 28s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
        +1 yarn tests 0m 26s Tests passed in hadoop-yarn-api.
            38m 38s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12730378/YARN-3523.20150505-1.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / 318081c
        hadoop-yarn-api test log https://builds.apache.org/job/PreCommit-YARN-Build/7703/artifact/patchprocess/testrun_hadoop-yarn-api.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/7703/testReport/
        Java 1.7.0_55
        uname Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/7703/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 15m 12s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac 7m 53s There were no new javac warning messages. +1 javadoc 10m 0s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 1m 6s There were no new checkstyle issues. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 install 1m 35s mvn install still works. +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse. +1 findbugs 1m 28s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 yarn tests 0m 26s Tests passed in hadoop-yarn-api.     38m 38s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12730378/YARN-3523.20150505-1.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 318081c hadoop-yarn-api test log https://builds.apache.org/job/PreCommit-YARN-Build/7703/artifact/patchprocess/testrun_hadoop-yarn-api.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/7703/testReport/ Java 1.7.0_55 uname Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-YARN-Build/7703/console This message was automatically generated.
        Hide
        djp Junping Du added a comment -

        Latest patch LGTM. However, I think we don't need @Private for every method because we already specified it in class level. In most cases, we need some different visibility in method level is to override the class level. Vinod Kumar Vavilapalli, what do you think?

        Show
        djp Junping Du added a comment - Latest patch LGTM. However, I think we don't need @Private for every method because we already specified it in class level. In most cases, we need some different visibility in method level is to override the class level. Vinod Kumar Vavilapalli , what do you think?
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        thanks for the review Junping Du,
        Well my view is to have @Private at every method to avoid in future similar mistake as currently happened. Thoughts ?

        Show
        Naganarasimha Naganarasimha G R added a comment - thanks for the review Junping Du , Well my view is to have @Private at every method to avoid in future similar mistake as currently happened. Thoughts ?
        Hide
        djp Junping Du added a comment -

        Well my view is to have @Private at every method to avoid in future similar mistake as currently happened. Thoughts?

        OK. This should be fine also.
        +1. Patch LGTM.

        Show
        djp Junping Du added a comment - Well my view is to have @Private at every method to avoid in future similar mistake as currently happened. Thoughts? OK. This should be fine also. +1. Patch LGTM.
        Hide
        djp Junping Du added a comment -

        I have commit latest patch to trunk and branch-2. Thanks Naganarasimha G R for contributing the patch, and review comments from Vinod Kumar Vavilapalli and Wangda Tan!

        Show
        djp Junping Du added a comment - I have commit latest patch to trunk and branch-2. Thanks Naganarasimha G R for contributing the patch, and review comments from Vinod Kumar Vavilapalli and Wangda Tan !
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-trunk-Commit #7758 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7758/)
        YARN-3523. Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java
        • hadoop-yarn-project/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #7758 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7758/ ) YARN-3523 . Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java hadoop-yarn-project/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #187 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/187/)
        YARN-3523. Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java
        • hadoop-yarn-project/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #187 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/187/ ) YARN-3523 . Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java hadoop-yarn-project/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Hadoop-Mapreduce-trunk #2136 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2136/)
        YARN-3523. Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9)

        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Mapreduce-trunk #2136 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2136/ ) YARN-3523 . Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9) hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        thanks for reviewing and committing the patch, Junping Du, Vinod Kumar Vavilapalli & Li Lu

        Show
        Naganarasimha Naganarasimha G R added a comment - thanks for reviewing and committing the patch, Junping Du , Vinod Kumar Vavilapalli & Li Lu
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #188 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/188/)
        YARN-3523. Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java
        • hadoop-yarn-project/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #188 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/188/ ) YARN-3523 . Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java hadoop-yarn-project/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #178 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/178/)
        YARN-3523. Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java
        • hadoop-yarn-project/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #178 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/178/ ) YARN-3523 . Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java hadoop-yarn-project/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Hadoop-Yarn-trunk #921 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/921/)
        YARN-3523. Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java
        • hadoop-yarn-project/CHANGES.txt
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Yarn-trunk #921 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/921/ ) YARN-3523 . Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java hadoop-yarn-project/CHANGES.txt
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk #2119 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2119/)
        YARN-3523. Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java
        • hadoop-yarn-project/CHANGES.txt
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk #2119 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2119/ ) YARN-3523 . Cleanup ResourceManagerAdministrationProtocol interface audience. Contributed by Naganarasimha G R (junping_du: rev 8e991f4b1d7226fdcd75c5dc9fe6e5ce721679b9) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/server/api/ResourceManagerAdministrationProtocol.java hadoop-yarn-project/CHANGES.txt

          People

          • Assignee:
            Naganarasimha Naganarasimha G R
            Reporter:
            leftnoteasy Wangda Tan
          • Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development