You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@slider.apache.org by te...@apache.org on 2014/10/14 01:49:23 UTC

[1/2] git commit: SLIDER-510 Correctly handle null check for yarnRegistryOperations in SliderAppMaster#unregisterComponent()

Repository: incubator-slider
Updated Branches:
  refs/heads/develop 6c4374392 -> 3ed76a5cc


SLIDER-510 Correctly handle null check for yarnRegistryOperations in SliderAppMaster#unregisterComponent()


Project: http://git-wip-us.apache.org/repos/asf/incubator-slider/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-slider/commit/28505391
Tree: http://git-wip-us.apache.org/repos/asf/incubator-slider/tree/28505391
Diff: http://git-wip-us.apache.org/repos/asf/incubator-slider/diff/28505391

Branch: refs/heads/develop
Commit: 285053911736b8948b3f4477af4f4558f85caa76
Parents: eba79a4
Author: tedyu <yu...@gmail.com>
Authored: Mon Oct 13 16:48:59 2014 -0700
Committer: tedyu <yu...@gmail.com>
Committed: Mon Oct 13 16:48:59 2014 -0700

----------------------------------------------------------------------
 .../java/org/apache/slider/server/appmaster/SliderAppMaster.java  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-slider/blob/28505391/slider-core/src/main/java/org/apache/slider/server/appmaster/SliderAppMaster.java
----------------------------------------------------------------------
diff --git a/slider-core/src/main/java/org/apache/slider/server/appmaster/SliderAppMaster.java b/slider-core/src/main/java/org/apache/slider/server/appmaster/SliderAppMaster.java
index a887cf8..d2a8936 100644
--- a/slider-core/src/main/java/org/apache/slider/server/appmaster/SliderAppMaster.java
+++ b/slider-core/src/main/java/org/apache/slider/server/appmaster/SliderAppMaster.java
@@ -1087,9 +1087,10 @@ public class SliderAppMaster extends AbstractSliderLaunchedService
    */
   public void unregisterComponent(ContainerId id) {
     log.info("Unregistering component {}", id);
-    if (yarnRegistryOperations== null) {
+    if (yarnRegistryOperations == null) {
       log.warn("Processing unregister component event before initialization " +
                "completed; init flag =" + initCompleted);
+      return;
     }
     String cid = RegistryPathUtils.encodeYarnID(id.toString());
     try {


[2/2] git commit: Merge branch 'develop' of https://git-wip-us.apache.org/repos/asf/incubator-slider into develop

Posted by te...@apache.org.
Merge branch 'develop' of https://git-wip-us.apache.org/repos/asf/incubator-slider into develop


Project: http://git-wip-us.apache.org/repos/asf/incubator-slider/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-slider/commit/3ed76a5c
Tree: http://git-wip-us.apache.org/repos/asf/incubator-slider/tree/3ed76a5c
Diff: http://git-wip-us.apache.org/repos/asf/incubator-slider/diff/3ed76a5c

Branch: refs/heads/develop
Commit: 3ed76a5cc6db20fd04b21a9fd4fb26e6e2a342e7
Parents: 2850539 6c43743
Author: tedyu <yu...@gmail.com>
Authored: Mon Oct 13 16:49:18 2014 -0700
Committer: tedyu <yu...@gmail.com>
Committed: Mon Oct 13 16:49:18 2014 -0700

----------------------------------------------------------------------
 .../funtest/accumulo/AccumuloReadWriteIT.groovy |  4 +-
 .../accumulo/AccumuloReadWriteSSLIT.groovy      |  4 +-
 .../memcached-win/package/scripts/memcached.py  |  1 -
 .../memcached/package/scripts/memcached.py      |  1 -
 .../storm-win/package/scripts/service.py        |  1 -
 app-packages/storm/package/scripts/service.py   |  9 +--
 .../core/providers/system.py                    | 28 +++++----
 .../TestDirectoryResource.py                    |  8 +--
 .../resource_management/TestFileResource.py     | 10 +--
 .../org/apache/slider/client/SliderClient.java  |  4 +-
 .../apache/slider/common/SliderXmlConfKeys.java |  2 +-
 .../slider/common/tools/ConfigHelper.java       | 20 +++++-
 .../utility/AbstractSliderLaunchedService.java  | 13 ++--
 .../slider/common/tools/TestConfigHelper.groovy | 65 ++++++++++++++++++++
 .../common/tools/TestConfigHelperHDFS.groovy    | 20 ------
 .../test/YarnZKMiniClusterTestBase.groovy       |  2 +-
 .../agent/TestAgentProviderService.java         | 13 ++--
 .../funtest/framework/CommandTestBase.groovy    |  5 +-
 .../funtest/basic/ClusterConnectivityIT.groovy  |  4 +-
 .../lifecycle/ClusterBuildDestroyIT.groovy      |  4 +-
 .../accumulo/funtest/AccumuloCIIT.groovy        |  4 +-
 .../funtest/FunctionalHBaseClusterIT.groovy     |  4 +-
 .../funtest/HBaseClusterBuildDestroyIT.groovy   |  4 +-
 23 files changed, 149 insertions(+), 81 deletions(-)
----------------------------------------------------------------------