Skip to content
Snippets Groups Projects
Commit 3128679a authored by buczekp's avatar buczekp
Browse files

[LMS-2029] minor: cleanup

SVN: 19819
parent bae34919
No related branches found
No related tags found
No related merge requests found
...@@ -26,7 +26,6 @@ import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDAOFactory; ...@@ -26,7 +26,6 @@ import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDAOFactory;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDataSetTypeDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDataSetTypeDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDataStoreDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDataStoreDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDatabaseInstanceDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDatabaseInstanceDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDynamicPropertyEvaluationScheduler;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IEntityPropertyTypeDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IEntityPropertyTypeDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IEntityTypeDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IEntityTypeDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IEventDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IEventDAO;
...@@ -35,7 +34,6 @@ import ch.systemsx.cisd.openbis.generic.server.dataaccess.IExternalDataDAO; ...@@ -35,7 +34,6 @@ import ch.systemsx.cisd.openbis.generic.server.dataaccess.IExternalDataDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IFileFormatTypeDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IFileFormatTypeDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IGridCustomColumnDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IGridCustomColumnDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IGridCustomFilterDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IGridCustomFilterDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.ISpaceDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IHibernateSearchDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IHibernateSearchDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.ILocatorTypeDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.ILocatorTypeDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IMaterialDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IMaterialDAO;
...@@ -48,18 +46,18 @@ import ch.systemsx.cisd.openbis.generic.server.dataaccess.IRoleAssignmentDAO; ...@@ -48,18 +46,18 @@ import ch.systemsx.cisd.openbis.generic.server.dataaccess.IRoleAssignmentDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.ISampleDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.ISampleDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.ISampleTypeDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.ISampleTypeDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IScriptDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IScriptDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.ISpaceDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IVocabularyDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IVocabularyDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IVocabularyTermDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IVocabularyTermDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.PersistencyResources; import ch.systemsx.cisd.openbis.generic.server.dataaccess.PersistencyResources;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.db.ICodeSequenceDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.db.ICodeSequenceDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.db.IPermIdDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.db.IPermIdDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.db.search.IFullTextIndexUpdateScheduler;
import ch.systemsx.cisd.openbis.generic.server.util.GroupIdentifierHelper; import ch.systemsx.cisd.openbis.generic.server.util.GroupIdentifierHelper;
import ch.systemsx.cisd.openbis.generic.shared.basic.dto.Identifier; import ch.systemsx.cisd.openbis.generic.shared.basic.dto.Identifier;
import ch.systemsx.cisd.openbis.generic.shared.dto.DatabaseInstancePE; import ch.systemsx.cisd.openbis.generic.shared.dto.DatabaseInstancePE;
import ch.systemsx.cisd.openbis.generic.shared.dto.SpacePE;
import ch.systemsx.cisd.openbis.generic.shared.dto.PersonPE; import ch.systemsx.cisd.openbis.generic.shared.dto.PersonPE;
import ch.systemsx.cisd.openbis.generic.shared.dto.Session; import ch.systemsx.cisd.openbis.generic.shared.dto.Session;
import ch.systemsx.cisd.openbis.generic.shared.dto.SpacePE;
import ch.systemsx.cisd.openbis.generic.shared.dto.identifier.SpaceIdentifier; import ch.systemsx.cisd.openbis.generic.shared.dto.identifier.SpaceIdentifier;
import ch.systemsx.cisd.openbis.generic.shared.dto.properties.EntityKind; import ch.systemsx.cisd.openbis.generic.shared.dto.properties.EntityKind;
...@@ -151,7 +149,7 @@ abstract class AbstractBusinessObject implements IDAOFactory ...@@ -151,7 +149,7 @@ abstract class AbstractBusinessObject implements IDAOFactory
final String code = String.valueOf(entityKind.name().charAt(0)) + id; final String code = String.valueOf(entityKind.name().charAt(0)) + id;
return code; return code;
} }
/** /**
* Returns the perm ID of specified identifier or creates a new one if it is <code>null</code>. * Returns the perm ID of specified identifier or creates a new one if it is <code>null</code>.
*/ */
...@@ -330,13 +328,4 @@ abstract class AbstractBusinessObject implements IDAOFactory ...@@ -330,13 +328,4 @@ abstract class AbstractBusinessObject implements IDAOFactory
return daoFactory.getRelationshipTypeDAO(); return daoFactory.getRelationshipTypeDAO();
} }
public IDynamicPropertyEvaluationScheduler getDynamicPropertyEvaluationScheduler()
{
return daoFactory.getDynamicPropertyEvaluationScheduler();
}
public IFullTextIndexUpdateScheduler getFullTextIndexUpdateScheduler()
{
return daoFactory.getFullTextIndexUpdateScheduler();
}
} }
...@@ -18,7 +18,6 @@ package ch.systemsx.cisd.openbis.generic.server.dataaccess; ...@@ -18,7 +18,6 @@ package ch.systemsx.cisd.openbis.generic.server.dataaccess;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.db.ICodeSequenceDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.db.ICodeSequenceDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.db.IPermIdDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.db.IPermIdDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.db.search.IFullTextIndexUpdateScheduler;
import ch.systemsx.cisd.openbis.generic.shared.dto.properties.EntityKind; import ch.systemsx.cisd.openbis.generic.shared.dto.properties.EntityKind;
/** /**
...@@ -99,10 +98,4 @@ public interface IDAOFactory extends IAuthorizationDAOFactory ...@@ -99,10 +98,4 @@ public interface IDAOFactory extends IAuthorizationDAOFactory
/** Returns an implementation of {@link IAuthorizationGroupDAO}. */ /** Returns an implementation of {@link IAuthorizationGroupDAO}. */
public IAuthorizationGroupDAO getAuthorizationGroupDAO(); public IAuthorizationGroupDAO getAuthorizationGroupDAO();
/** Returns an implementation of {@link IDynamicPropertyEvaluationScheduler}. */
public IDynamicPropertyEvaluationScheduler getDynamicPropertyEvaluationScheduler();
/** Returns an implementation of {@link IFullTextIndexUpdateScheduler}. */
public IFullTextIndexUpdateScheduler getFullTextIndexUpdateScheduler();
} }
...@@ -59,7 +59,8 @@ public class DynamicPropertyEvaluationMaintenanceTask implements IMaintenanceTas ...@@ -59,7 +59,8 @@ public class DynamicPropertyEvaluationMaintenanceTask implements IMaintenanceTas
{ {
operationLog.info("execution started"); operationLog.info("execution started");
IDynamicPropertyEvaluationScheduler scheduler = IDynamicPropertyEvaluationScheduler scheduler =
CommonServiceProvider.getDAOFactory().getDynamicPropertyEvaluationScheduler(); CommonServiceProvider.getDAOFactory().getPersistencyResources()
.getDynamicPropertyEvaluationScheduler();
// all entities will be scheduled for update so previous schedule can be cleared // all entities will be scheduled for update so previous schedule can be cleared
scheduler.clear(); scheduler.clear();
......
...@@ -54,7 +54,6 @@ import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDAOFactory; ...@@ -54,7 +54,6 @@ import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDAOFactory;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDataSetTypeDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDataSetTypeDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDataStoreDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDataStoreDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDatabaseInstanceDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDatabaseInstanceDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IDynamicPropertyEvaluationScheduler;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IEntityTypeDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IEntityTypeDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IExperimentDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IExperimentDAO;
import ch.systemsx.cisd.openbis.generic.server.dataaccess.IExternalDataDAO; import ch.systemsx.cisd.openbis.generic.server.dataaccess.IExternalDataDAO;
...@@ -174,8 +173,6 @@ public abstract class AbstractServerTestCase extends AssertJUnit ...@@ -174,8 +173,6 @@ public abstract class AbstractServerTestCase extends AssertJUnit
protected IQueryDAO queryDAO; protected IQueryDAO queryDAO;
protected IDynamicPropertyEvaluationScheduler evaluator;
protected IPropertiesBatchManager propertiesBatchManager; protected IPropertiesBatchManager propertiesBatchManager;
@BeforeMethod @BeforeMethod
...@@ -226,8 +223,6 @@ public abstract class AbstractServerTestCase extends AssertJUnit ...@@ -226,8 +223,6 @@ public abstract class AbstractServerTestCase extends AssertJUnit
propertyTypeTable = context.mock(IPropertyTypeTable.class); propertyTypeTable = context.mock(IPropertyTypeTable.class);
materialTable = context.mock(IMaterialTable.class); materialTable = context.mock(IMaterialTable.class);
materialLister = context.mock(IMaterialLister.class); materialLister = context.mock(IMaterialLister.class);
//
evaluator = context.mock(IDynamicPropertyEvaluationScheduler.class);
homeDatabaseInstance = CommonTestUtils.createHomeDatabaseInstance(); homeDatabaseInstance = CommonTestUtils.createHomeDatabaseInstance();
context.checking(new Expectations() context.checking(new Expectations()
...@@ -261,8 +256,6 @@ public abstract class AbstractServerTestCase extends AssertJUnit ...@@ -261,8 +256,6 @@ public abstract class AbstractServerTestCase extends AssertJUnit
will(returnValue(dataSetTypeDAO)); will(returnValue(dataSetTypeDAO));
allowing(daoFactory).getDataStoreDAO(); allowing(daoFactory).getDataStoreDAO();
will(returnValue(dataStoreDAO)); will(returnValue(dataStoreDAO));
allowing(daoFactory).getDynamicPropertyEvaluationScheduler();
will(returnValue(evaluator));
} }
}); });
} }
......
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