Fix factory instance creation instability

Refactor the factory registry and instance creation.
Remove unnecessary static code in Factory creation.
Centralize the responsibility to read the factory registry configuration
and cache the factory instances in a Singleton called
FactoryRegistryManager.
Clean the base abstract factory that was sharing non factory code, it
had all the factory registry code statically, never used by the concrete
factory classes.

Change-Id: Ie2e9c29013acd36ddaf15383dd6b06432fbdbb66
Issue-ID: SDC-1905
Signed-off-by: andre.schmid <andre.schmid@est.tech>

1 files changed, 0 insertions, 61 deletions

diff --git a/openecomp-be/lib/openecomp-core-lib/openecomp-facade-lib/openecomp-facade-core/src/main/java/org/openecomp/core/factory/api/AbstractComponentFactory.java b/openecomp-be/lib/openecomp-core-lib/openecomp-facade-lib/openecomp-facade-core/src/main/java/org/openecomp/core/factory/api/AbstractComponentFactory.java
index 1ea6fe8d43..3de0d88f43 100644
--- a/openecomp-be/lib/openecomp-core-lib/openecomp-facade-lib/openecomp-facade-core/src/main/java/org/openecomp/core/factory/api/AbstractComponentFactory.java
+++ b/openecomp-be/lib/openecomp-core-lib/openecomp-facade-lib/openecomp-facade-core/src/main/java/org/openecomp/core/factory/api/AbstractComponentFactory.java
@@ -16,66 +16,5 @@
 
 package org.openecomp.core.factory.api;
 
-import org.apache.commons.lang3.StringUtils;
-import org.openecomp.core.factory.FactoryConfig;
-import org.openecomp.core.factory.impl.AbstractFactoryBase;
-import org.openecomp.sdc.common.errors.CoreException;
-import org.openecomp.sdc.common.errors.ErrorCategory;
-import org.openecomp.sdc.common.errors.ErrorCode;
-
-import java.util.Map;
-
 public abstract class AbstractComponentFactory<I> extends AbstractFactory<I> {
-
-  static {
-    Registry registry = new RegistryImpl();
-    InitializationHelper.registerFactoryMapping(registry);
-  }
-
-  @FunctionalInterface
-  interface Registry {
-    void register(String factory, String impl);
-  }
-
-  private static class RegistryImpl implements Registry {
-    @Override
-    public void register(String factory, String impl) {
-      AbstractFactoryBase.registerFactory(factory, impl);
-    }
-  }
-
-  static class InitializationHelper {
-
-
-    private static boolean isRegistered = false;
-
-    private InitializationHelper() {
-    }
-
-    static synchronized void registerFactoryMapping(Registry registry) {
-      if (!isRegistered) {
-        registerFactoryMappingImpl(registry);
-        isRegistered = true;
-      }
-    }
-
-    private static void registerFactoryMappingImpl(Registry registry) {
-      Map<String, String> factoryMap = FactoryConfig.getFactoriesMap();
-
-        for (Map.Entry<String, String> entry : factoryMap.entrySet()) {
-          String abstractClassName = entry.getKey();
-          String concreteTypeName = entry.getValue();
-
-          if (StringUtils.isEmpty(concreteTypeName)) {
-            throw new CoreException(
-                new ErrorCode.ErrorCodeBuilder().withId("E0003")
-                    .withMessage("Missing configuration value:" + concreteTypeName + ".")
-                    .withCategory(ErrorCategory.SYSTEM).build());
-
-          }
-          registry.register(abstractClassName, concreteTypeName);
-        }
-    }
-  }
-
 }