Skip to content
Snippets Groups Projects
Commit c45ca96e authored by brinn's avatar brinn
Browse files

Move IIntializable to resource package and rename IResource to IReleasable.

SVN: 27018
parent 0e2c04f5
No related branches found
No related tags found
No related merge requests found
Showing
with 26 additions and 26 deletions
...@@ -22,7 +22,7 @@ import org.python.core.PySystemState; ...@@ -22,7 +22,7 @@ import org.python.core.PySystemState;
import ch.systemsx.cisd.common.logging.LogCategory; import ch.systemsx.cisd.common.logging.LogCategory;
import ch.systemsx.cisd.common.logging.LogFactory; import ch.systemsx.cisd.common.logging.LogFactory;
import ch.systemsx.cisd.common.resource.IResource; import ch.systemsx.cisd.common.resource.IReleasable;
import ch.systemsx.cisd.common.resource.Resources; import ch.systemsx.cisd.common.resource.Resources;
/** /**
...@@ -49,9 +49,9 @@ public class PythonInterpreter extends org.python.util.PythonInterpreter ...@@ -49,9 +49,9 @@ public class PythonInterpreter extends org.python.util.PythonInterpreter
public void set(String name, Object value) public void set(String name, Object value)
{ {
super.set(name, value); super.set(name, value);
if (value instanceof IResource) if (value instanceof IReleasable)
{ {
resources.add((IResource) value); resources.add((IReleasable) value);
} }
} }
...@@ -59,9 +59,9 @@ public class PythonInterpreter extends org.python.util.PythonInterpreter ...@@ -59,9 +59,9 @@ public class PythonInterpreter extends org.python.util.PythonInterpreter
public void set(String name, PyObject value) public void set(String name, PyObject value)
{ {
super.set(name, value); super.set(name, value);
if (value instanceof IResource) if (value instanceof IReleasable)
{ {
resources.add((IResource) value); resources.add((IReleasable) value);
} }
} }
......
...@@ -14,7 +14,7 @@ ...@@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
package ch.systemsx.cisd.common.utilities; package ch.systemsx.cisd.common.resource;
/** /**
* Interface for initializable classes. * Interface for initializable classes.
......
...@@ -19,7 +19,7 @@ package ch.systemsx.cisd.common.resource; ...@@ -19,7 +19,7 @@ package ch.systemsx.cisd.common.resource;
/** /**
* @author pkupczyk * @author pkupczyk
*/ */
public interface IResource public interface IReleasable
{ {
public void release(); public void release();
......
...@@ -29,14 +29,14 @@ public class Resources ...@@ -29,14 +29,14 @@ public class Resources
private Logger logger; private Logger logger;
public Set<IResource> resources = new LinkedHashSet<IResource>(); public Set<IReleasable> resources = new LinkedHashSet<IReleasable>();
public Resources(Logger logger) public Resources(Logger logger)
{ {
this.logger = logger; this.logger = logger;
} }
public void add(IResource resource) public void add(IReleasable resource)
{ {
resources.add(resource); resources.add(resource);
} }
...@@ -45,7 +45,7 @@ public class Resources ...@@ -45,7 +45,7 @@ public class Resources
{ {
if (resources.size() > 0) if (resources.size() > 0)
{ {
for (IResource resource : resources) for (IReleasable resource : resources)
{ {
try try
{ {
......
...@@ -31,10 +31,10 @@ public class ResourcesTest ...@@ -31,10 +31,10 @@ public class ResourcesTest
{ {
Mockery context = new Mockery(); Mockery context = new Mockery();
final IResource failingResource1 = context.mock(IResource.class, "failingResource1"); final IReleasable failingResource1 = context.mock(IReleasable.class, "failingResource1");
final IResource failingResource2 = context.mock(IResource.class, "failingResource2"); final IReleasable failingResource2 = context.mock(IReleasable.class, "failingResource2");
final IResource notFailingResource1 = context.mock(IResource.class, "notFailingResource1"); final IReleasable notFailingResource1 = context.mock(IReleasable.class, "notFailingResource1");
final IResource notFailingResource2 = context.mock(IResource.class, "notFailingResource2"); final IReleasable notFailingResource2 = context.mock(IReleasable.class, "notFailingResource2");
Resources resources = new Resources(null); Resources resources = new Resources(null);
resources.add(notFailingResource1); resources.add(notFailingResource1);
...@@ -67,7 +67,7 @@ public class ResourcesTest ...@@ -67,7 +67,7 @@ public class ResourcesTest
{ {
Mockery context = new Mockery(); Mockery context = new Mockery();
final IResource resource = context.mock(IResource.class); final IReleasable resource = context.mock(IReleasable.class);
Resources resources = new Resources(null); Resources resources = new Resources(null);
resources.add(resource); resources.add(resource);
...@@ -90,7 +90,7 @@ public class ResourcesTest ...@@ -90,7 +90,7 @@ public class ResourcesTest
{ {
Mockery context = new Mockery(); Mockery context = new Mockery();
final IResource resource = context.mock(IResource.class); final IReleasable resource = context.mock(IReleasable.class);
Resources resources = new Resources(null); Resources resources = new Resources(null);
resources.add(resource); resources.add(resource);
......
...@@ -27,7 +27,7 @@ import ch.systemsx.cisd.common.logging.LogCategory; ...@@ -27,7 +27,7 @@ import ch.systemsx.cisd.common.logging.LogCategory;
import ch.systemsx.cisd.common.logging.LogFactory; import ch.systemsx.cisd.common.logging.LogFactory;
import ch.systemsx.cisd.common.mail.IMailClient; import ch.systemsx.cisd.common.mail.IMailClient;
import ch.systemsx.cisd.common.properties.PropertyUtils; import ch.systemsx.cisd.common.properties.PropertyUtils;
import ch.systemsx.cisd.common.resource.IResource; import ch.systemsx.cisd.common.resource.IReleasable;
import ch.systemsx.cisd.common.shared.basic.utils.StringUtils; import ch.systemsx.cisd.common.shared.basic.utils.StringUtils;
import ch.systemsx.cisd.etlserver.registrator.recovery.IDataSetStorageRecoveryManager; import ch.systemsx.cisd.etlserver.registrator.recovery.IDataSetStorageRecoveryManager;
import ch.systemsx.cisd.etlserver.validation.IDataSetValidator; import ch.systemsx.cisd.etlserver.validation.IDataSetValidator;
...@@ -40,7 +40,7 @@ import ch.systemsx.cisd.openbis.generic.shared.basic.dto.Person; ...@@ -40,7 +40,7 @@ import ch.systemsx.cisd.openbis.generic.shared.basic.dto.Person;
* *
* @author Chandrasekhar Ramakrishnan * @author Chandrasekhar Ramakrishnan
*/ */
public class TopLevelDataSetRegistratorGlobalState implements IResource public class TopLevelDataSetRegistratorGlobalState implements IReleasable
{ {
// can be used from dropboxes // can be used from dropboxes
public static final Logger operationLog = LogFactory.getLogger(LogCategory.OPERATION, public static final Logger operationLog = LogFactory.getLogger(LogCategory.OPERATION,
...@@ -407,9 +407,9 @@ public class TopLevelDataSetRegistratorGlobalState implements IResource ...@@ -407,9 +407,9 @@ public class TopLevelDataSetRegistratorGlobalState implements IResource
@Override @Override
public void release() public void release()
{ {
if (getDataSourceQueryService() instanceof IResource) if (getDataSourceQueryService() instanceof IReleasable)
{ {
((IResource) getDataSourceQueryService()).release(); ((IReleasable) getDataSourceQueryService()).release();
} }
} }
......
...@@ -65,8 +65,8 @@ import ch.systemsx.cisd.common.exceptions.EnvironmentFailureException; ...@@ -65,8 +65,8 @@ import ch.systemsx.cisd.common.exceptions.EnvironmentFailureException;
import ch.systemsx.cisd.common.logging.LogCategory; import ch.systemsx.cisd.common.logging.LogCategory;
import ch.systemsx.cisd.common.logging.LogFactory; import ch.systemsx.cisd.common.logging.LogFactory;
import ch.systemsx.cisd.common.logging.LogInitializer; import ch.systemsx.cisd.common.logging.LogInitializer;
import ch.systemsx.cisd.common.resource.IInitializable;
import ch.systemsx.cisd.common.utilities.ExtendedProperties; import ch.systemsx.cisd.common.utilities.ExtendedProperties;
import ch.systemsx.cisd.common.utilities.IInitializable;
import ch.systemsx.cisd.openbis.dss.generic.shared.IEncapsulatedOpenBISService; import ch.systemsx.cisd.openbis.dss.generic.shared.IEncapsulatedOpenBISService;
import ch.systemsx.cisd.openbis.dss.generic.shared.ServiceProvider; import ch.systemsx.cisd.openbis.dss.generic.shared.ServiceProvider;
import ch.systemsx.cisd.openbis.dss.generic.shared.api.internal.authorization.DssSessionAuthorizationHolder; import ch.systemsx.cisd.openbis.dss.generic.shared.api.internal.authorization.DssSessionAuthorizationHolder;
......
...@@ -22,9 +22,9 @@ import java.util.Map; ...@@ -22,9 +22,9 @@ import java.util.Map;
import org.apache.log4j.Logger; import org.apache.log4j.Logger;
import ch.systemsx.cisd.common.exceptions.InvalidAuthenticationException; import ch.systemsx.cisd.common.exceptions.InvalidAuthenticationException;
import ch.systemsx.cisd.common.resource.IInitializable;
import ch.systemsx.cisd.common.serviceconversation.ServiceMessage; import ch.systemsx.cisd.common.serviceconversation.ServiceMessage;
import ch.systemsx.cisd.common.spring.IInvocationLoggerContext; import ch.systemsx.cisd.common.spring.IInvocationLoggerContext;
import ch.systemsx.cisd.common.utilities.IInitializable;
import ch.systemsx.cisd.openbis.generic.shared.IDataStoreService; import ch.systemsx.cisd.openbis.generic.shared.IDataStoreService;
import ch.systemsx.cisd.openbis.generic.shared.basic.dto.CustomImportFile; import ch.systemsx.cisd.openbis.generic.shared.basic.dto.CustomImportFile;
import ch.systemsx.cisd.openbis.generic.shared.basic.dto.ExternalData; import ch.systemsx.cisd.openbis.generic.shared.basic.dto.ExternalData;
......
...@@ -28,14 +28,14 @@ import org.apache.log4j.Logger; ...@@ -28,14 +28,14 @@ import org.apache.log4j.Logger;
import ch.systemsx.cisd.common.logging.LogCategory; import ch.systemsx.cisd.common.logging.LogCategory;
import ch.systemsx.cisd.common.logging.LogFactory; import ch.systemsx.cisd.common.logging.LogFactory;
import ch.systemsx.cisd.common.resource.IResource; import ch.systemsx.cisd.common.resource.IReleasable;
import ch.systemsx.cisd.common.resource.Resources; import ch.systemsx.cisd.common.resource.Resources;
import ch.systemsx.cisd.openbis.dss.generic.shared.api.internal.v1.IDataSourceQueryService; import ch.systemsx.cisd.openbis.dss.generic.shared.api.internal.v1.IDataSourceQueryService;
/** /**
* @author Chandrasekhar Ramakrishnan * @author Chandrasekhar Ramakrishnan
*/ */
public class DataSourceQueryService implements IDataSourceQueryService, IResource public class DataSourceQueryService implements IDataSourceQueryService, IReleasable
{ {
private static final Logger operationLog = LogFactory.getLogger(LogCategory.OPERATION, private static final Logger operationLog = LogFactory.getLogger(LogCategory.OPERATION,
DataSourceQueryService.class); DataSourceQueryService.class);
...@@ -82,7 +82,7 @@ public class DataSourceQueryService implements IDataSourceQueryService, IResourc ...@@ -82,7 +82,7 @@ public class DataSourceQueryService implements IDataSourceQueryService, IResourc
resources.clear(); resources.clear();
} }
private static class DataSetResource implements IResource private static class DataSetResource implements IReleasable
{ {
private DataSet<?> dataSet; private DataSet<?> dataSet;
......
...@@ -16,7 +16,7 @@ ...@@ -16,7 +16,7 @@
package ch.systemsx.cisd.openbis.dss.generic.shared; package ch.systemsx.cisd.openbis.dss.generic.shared;
import ch.systemsx.cisd.common.utilities.IInitializable; import ch.systemsx.cisd.common.resource.IInitializable;
import ch.systemsx.cisd.openbis.generic.shared.IDataStoreService; import ch.systemsx.cisd.openbis.generic.shared.IDataStoreService;
/** /**
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment