diff --git a/dhp-common/src/main/java/eu/dnetlib/dhp/collector/worker/model/ApiDescriptor.java b/dhp-common/src/main/java/eu/dnetlib/dhp/collection/ApiDescriptor.java
similarity index 93%
rename from dhp-common/src/main/java/eu/dnetlib/dhp/collector/worker/model/ApiDescriptor.java
rename to dhp-common/src/main/java/eu/dnetlib/dhp/collection/ApiDescriptor.java
index 8ba30faeb..12937a197 100644
--- a/dhp-common/src/main/java/eu/dnetlib/dhp/collector/worker/model/ApiDescriptor.java
+++ b/dhp-common/src/main/java/eu/dnetlib/dhp/collection/ApiDescriptor.java
@@ -1,5 +1,5 @@
-package eu.dnetlib.dhp.collector.worker.model;
+package eu.dnetlib.dhp.collection;
import java.util.HashMap;
import java.util.Map;
diff --git a/dhp-common/src/main/java/eu/dnetlib/dhp/common/Constants.java b/dhp-common/src/main/java/eu/dnetlib/dhp/common/Constants.java
index 2b8ef4e30..eb4cb91ed 100644
--- a/dhp-common/src/main/java/eu/dnetlib/dhp/common/Constants.java
+++ b/dhp-common/src/main/java/eu/dnetlib/dhp/common/Constants.java
@@ -27,4 +27,24 @@ public class Constants {
coarCodeLabelMap.put("c_f1cf", "EMBARGO");
}
+ public static final String SEQUENCE_FILE_NAME = "/sequence_file";
+ public static final String REPORT_FILE_NAME = "/report";
+ public static final String MDSTORE_DATA_PATH = "/store";
+ public static final String MDSTORE_SIZE_PATH = "/size";
+
+ public static final String COLLECTION_MODE = "collectionMode";
+ public static final String METADATA_ENCODING = "metadataEncoding";
+ public static final String OOZIE_WF_PATH = "oozieWfPath";
+ public static final String DNET_MESSAGE_MGR_URL = "dnetMessageManagerURL";
+
+ public static final String MAX_NUMBER_OF_RETRY = "maxNumberOfRetry";
+ public static final String REQUEST_DELAY = "requestDelay";
+ public static final String RETRY_DELAY = "retryDelay";
+ public static final String CONNECT_TIMEOUT = "connectTimeOut";
+ public static final String READ_TIMEOUT = "readTimeOut";
+
+ public static final String CONTENT_TOTALITEMS = "TotalItems";
+ public static final String CONTENT_INVALIDRECORDS = "InvalidRecords";
+ public static final String CONTENT_TRANSFORMEDRECORDS = "transformedItems";
+
}
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/aggregation/common/AggregationConstants.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/aggregation/common/AggregationConstants.java
deleted file mode 100644
index 8e0b7260d..000000000
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/aggregation/common/AggregationConstants.java
+++ /dev/null
@@ -1,15 +0,0 @@
-
-package eu.dnetlib.dhp.aggregation.common;
-
-public class AggregationConstants {
-
- public static final String SEQUENCE_FILE_NAME = "/sequence_file";
- public static final String REPORT_FILE_NAME = "/report";
- public static final String MDSTORE_DATA_PATH = "/store";
- public static final String MDSTORE_SIZE_PATH = "/size";
-
- public static final String CONTENT_TOTALITEMS = "TotalItems";
- public static final String CONTENT_INVALIDRECORDS = "InvalidRecords";
- public static final String CONTENT_TRANSFORMEDRECORDS = "transformedItems";
-
-}
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/aggregation/mdstore/MDStoreActionNode.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/aggregation/mdstore/MDStoreActionNode.java
index 9a47a1d66..829921dd8 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/aggregation/mdstore/MDStoreActionNode.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/aggregation/mdstore/MDStoreActionNode.java
@@ -1,8 +1,7 @@
package eu.dnetlib.dhp.aggregation.mdstore;
-import static eu.dnetlib.dhp.aggregation.common.AggregationConstants.*;
-import static eu.dnetlib.dhp.application.ApplicationUtils.*;
+import static eu.dnetlib.dhp.common.Constants.*;
import static eu.dnetlib.dhp.utils.DHPUtils.*;
import java.net.URI;
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/GenerateNativeStoreSparkJob.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/GenerateNativeStoreSparkJob.java
index 5c24bb7ec..ee82cc94f 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/GenerateNativeStoreSparkJob.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/GenerateNativeStoreSparkJob.java
@@ -1,7 +1,7 @@
package eu.dnetlib.dhp.collection;
-import static eu.dnetlib.dhp.aggregation.common.AggregationConstants.*;
+import static eu.dnetlib.dhp.common.Constants.*;
import static eu.dnetlib.dhp.common.SparkSessionSupport.runWithSparkSession;
import static eu.dnetlib.dhp.utils.DHPUtils.*;
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/CollectorPlugin.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/CollectorPlugin.java
index 614aa4e69..e2be481ed 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/CollectorPlugin.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/CollectorPlugin.java
@@ -3,9 +3,9 @@ package eu.dnetlib.dhp.collection.plugin;
import java.util.stream.Stream;
+import eu.dnetlib.dhp.collection.ApiDescriptor;
import eu.dnetlib.dhp.collection.worker.CollectorException;
import eu.dnetlib.dhp.collection.worker.CollectorPluginReport;
-import eu.dnetlib.dhp.collector.worker.model.ApiDescriptor;
public interface CollectorPlugin {
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/oai/OaiCollectorPlugin.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/oai/OaiCollectorPlugin.java
index 7ec2f09be..84228abf4 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/oai/OaiCollectorPlugin.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/plugin/oai/OaiCollectorPlugin.java
@@ -13,11 +13,11 @@ import com.google.common.base.Splitter;
import com.google.common.collect.Iterators;
import com.google.common.collect.Lists;
+import eu.dnetlib.dhp.collection.ApiDescriptor;
import eu.dnetlib.dhp.collection.plugin.CollectorPlugin;
import eu.dnetlib.dhp.collection.worker.CollectorException;
import eu.dnetlib.dhp.collection.worker.CollectorPluginReport;
import eu.dnetlib.dhp.collection.worker.HttpClientParams;
-import eu.dnetlib.dhp.collector.worker.model.ApiDescriptor;
public class OaiCollectorPlugin implements CollectorPlugin {
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorker.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorker.java
index 71dee0d03..c2d32019d 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorker.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorker.java
@@ -1,7 +1,7 @@
package eu.dnetlib.dhp.collection.worker;
-import static eu.dnetlib.dhp.aggregation.common.AggregationConstants.SEQUENCE_FILE_NAME;
+import static eu.dnetlib.dhp.common.Constants.SEQUENCE_FILE_NAME;
import java.io.IOException;
import java.util.concurrent.atomic.AtomicInteger;
@@ -15,8 +15,8 @@ import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import eu.dnetlib.data.mdstore.manager.common.model.MDStoreVersion;
+import eu.dnetlib.dhp.collection.ApiDescriptor;
import eu.dnetlib.dhp.collection.plugin.CollectorPlugin;
-import eu.dnetlib.dhp.collector.worker.model.ApiDescriptor;
import eu.dnetlib.dhp.message.MessageSender;
public class CollectorWorker {
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorkerApplication.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorkerApplication.java
index a6c254d42..6e4237bee 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorkerApplication.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorkerApplication.java
@@ -1,7 +1,7 @@
package eu.dnetlib.dhp.collection.worker;
-import static eu.dnetlib.dhp.aggregation.common.AggregationConstants.*;
+import static eu.dnetlib.dhp.common.Constants.*;
import static eu.dnetlib.dhp.utils.DHPUtils.*;
import java.io.IOException;
@@ -17,7 +17,7 @@ import org.slf4j.LoggerFactory;
import eu.dnetlib.data.mdstore.manager.common.model.MDStoreVersion;
import eu.dnetlib.dhp.application.ArgumentApplicationParser;
-import eu.dnetlib.dhp.collector.worker.model.ApiDescriptor;
+import eu.dnetlib.dhp.collection.ApiDescriptor;
import eu.dnetlib.dhp.message.MessageSender;
/**
@@ -55,7 +55,7 @@ public class CollectorWorkerApplication {
final String mdStoreVersion = argumentParser.get("mdStoreVersion");
log.info("mdStoreVersion is {}", mdStoreVersion);
- final String dnetMessageManagerURL = argumentParser.get("dnetMessageManagerURL");
+ final String dnetMessageManagerURL = argumentParser.get(DNET_MESSAGE_MGR_URL);
log.info("dnetMessageManagerURL is {}", dnetMessageManagerURL);
final String workflowId = argumentParser.get("workflowId");
@@ -87,15 +87,23 @@ public class CollectorWorkerApplication {
clientParams
.setMaxNumberOfRetry(
Optional
- .ofNullable(argumentParser.get("maxNumberOfRetry"))
+ .ofNullable(argumentParser.get(MAX_NUMBER_OF_RETRY))
.map(Integer::parseInt)
.orElse(HttpClientParams._maxNumberOfRetry));
log.info("maxNumberOfRetry is {}", clientParams.getMaxNumberOfRetry());
+ clientParams
+ .setRequestDelay(
+ Optional
+ .ofNullable(argumentParser.get(REQUEST_DELAY))
+ .map(Integer::parseInt)
+ .orElse(HttpClientParams._requestDelay));
+ log.info("requestDelay is {}", clientParams.getRequestDelay());
+
clientParams
.setRetryDelay(
Optional
- .ofNullable(argumentParser.get("retryDelay"))
+ .ofNullable(argumentParser.get(RETRY_DELAY))
.map(Integer::parseInt)
.orElse(HttpClientParams._retryDelay));
log.info("retryDelay is {}", clientParams.getRetryDelay());
@@ -103,7 +111,7 @@ public class CollectorWorkerApplication {
clientParams
.setConnectTimeOut(
Optional
- .ofNullable(argumentParser.get("connectTimeOut"))
+ .ofNullable(argumentParser.get(CONNECT_TIMEOUT))
.map(Integer::parseInt)
.orElse(HttpClientParams._connectTimeOut));
log.info("connectTimeOut is {}", clientParams.getConnectTimeOut());
@@ -111,7 +119,7 @@ public class CollectorWorkerApplication {
clientParams
.setReadTimeOut(
Optional
- .ofNullable(argumentParser.get("readTimeOut"))
+ .ofNullable(argumentParser.get(READ_TIMEOUT))
.map(Integer::parseInt)
.orElse(HttpClientParams._readTimeOut));
log.info("readTimeOut is {}", clientParams.getReadTimeOut());
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorkerReporter.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorkerReporter.java
index 3a8145946..3f6fc4784 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorkerReporter.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/CollectorWorkerReporter.java
@@ -1,7 +1,7 @@
package eu.dnetlib.dhp.collection.worker;
-import static eu.dnetlib.dhp.aggregation.common.AggregationConstants.REPORT_FILE_NAME;
+import static eu.dnetlib.dhp.common.Constants.REPORT_FILE_NAME;
import static eu.dnetlib.dhp.utils.DHPUtils.*;
import java.io.IOException;
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/HttpClientParams.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/HttpClientParams.java
index 315dd27c2..f45790460 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/HttpClientParams.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/HttpClientParams.java
@@ -6,22 +6,46 @@ package eu.dnetlib.dhp.collection.worker;
*/
public class HttpClientParams {
+ // Defaults
public static int _maxNumberOfRetry = 3;
+ public static int _requestDelay = 0; // milliseconds
public static int _retryDelay = 10; // seconds
public static int _connectTimeOut = 10; // seconds
public static int _readTimeOut = 30; // seconds
+ /**
+ * Maximum number of allowed retires before failing
+ */
private int maxNumberOfRetry;
+
+ /**
+ * Delay between request (Milliseconds)
+ */
+ private int requestDelay;
+
+ /**
+ * Time to wait after a failure before retrying (Seconds)
+ */
private int retryDelay;
+
+ /**
+ * Connect timeout (Seconds)
+ */
private int connectTimeOut;
+
+ /**
+ * Read timeout (Seconds)
+ */
private int readTimeOut;
public HttpClientParams() {
- this(_maxNumberOfRetry, _retryDelay, _connectTimeOut, _readTimeOut);
+ this(_maxNumberOfRetry, _requestDelay, _retryDelay, _connectTimeOut, _readTimeOut);
}
- public HttpClientParams(int maxNumberOfRetry, int retryDelay, int connectTimeOut, int readTimeOut) {
+ public HttpClientParams(int maxNumberOfRetry, int requestDelay, int retryDelay, int connectTimeOut,
+ int readTimeOut) {
this.maxNumberOfRetry = maxNumberOfRetry;
+ this.requestDelay = requestDelay;
this.retryDelay = retryDelay;
this.connectTimeOut = connectTimeOut;
this.readTimeOut = readTimeOut;
@@ -35,6 +59,14 @@ public class HttpClientParams {
this.maxNumberOfRetry = maxNumberOfRetry;
}
+ public int getRequestDelay() {
+ return requestDelay;
+ }
+
+ public void setRequestDelay(int requestDelay) {
+ this.requestDelay = requestDelay;
+ }
+
public int getRetryDelay() {
return retryDelay;
}
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/HttpConnector2.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/HttpConnector2.java
index ee3acf432..368c89509 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/HttpConnector2.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/collection/worker/HttpConnector2.java
@@ -18,7 +18,7 @@ import org.slf4j.LoggerFactory;
/**
* Migrated from https://svn.driver.research-infrastructures.eu/driver/dnet45/modules/dnet-modular-collector-service/trunk/src/main/java/eu/dnetlib/data/collector/plugins/HttpConnector.java
*
- * @author jochen, michele, andrea, alessia
+ * @author jochen, michele, andrea, alessia, claudio
*/
public class HttpConnector2 {
@@ -83,14 +83,22 @@ public class HttpConnector2 {
final CollectorPluginReport report) throws CollectorException, IOException {
if (retryNumber > getClientParams().getMaxNumberOfRetry()) {
- throw new CollectorException("Max number of retries exceeded. Cause: \n " + report);
+ final String msg = String
+ .format(
+ "Max number of retries (%s/%s) exceeded, failing.",
+ retryNumber, getClientParams().getMaxNumberOfRetry());
+ log.error(msg);
+ throw new CollectorException(msg);
}
- log.info("Downloading attempt {} [{}]", retryNumber, requestUrl);
+ log.info("Request attempt {} [{}]", retryNumber, requestUrl);
InputStream input = null;
try {
+ if (getClientParams().getRequestDelay() > 0) {
+ backoffAndSleep(getClientParams().getRequestDelay());
+ }
final HttpURLConnection urlConn = (HttpURLConnection) new URL(requestUrl).openConnection();
urlConn.setInstanceFollowRedirects(false);
urlConn.setReadTimeout(getClientParams().getReadTimeOut() * 1000);
@@ -190,10 +198,10 @@ public class HttpConnector2 {
}
}
- private void backoffAndSleep(int sleepTime) throws CollectorException {
- log.info("I'm going to sleep for {}ms", sleepTime);
+ private void backoffAndSleep(int sleepTimeMs) throws CollectorException {
+ log.info("I'm going to sleep for {}ms", sleepTimeMs);
try {
- Thread.sleep(sleepTime);
+ Thread.sleep(sleepTimeMs);
} catch (InterruptedException e) {
log.error(e.getMessage(), e);
throw new CollectorException(e);
diff --git a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/transformation/TransformSparkJobNode.java b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/transformation/TransformSparkJobNode.java
index c0a03e081..e628e7645 100644
--- a/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/transformation/TransformSparkJobNode.java
+++ b/dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/transformation/TransformSparkJobNode.java
@@ -1,7 +1,7 @@
package eu.dnetlib.dhp.transformation;
-import static eu.dnetlib.dhp.aggregation.common.AggregationConstants.*;
+import static eu.dnetlib.dhp.common.Constants.*;
import static eu.dnetlib.dhp.common.SparkSessionSupport.runWithSparkSession;
import static eu.dnetlib.dhp.utils.DHPUtils.*;
diff --git a/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/collection/collector_worker_input_parameter.json b/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/collection/collector_worker_input_parameter.json
index f3eaf2d71..cd4b8224b 100644
--- a/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/collection/collector_worker_input_parameter.json
+++ b/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/collection/collector_worker_input_parameter.json
@@ -30,25 +30,31 @@
"paramRequired": true
},
{
- "paramName": "mr",
+ "paramName": "mnr",
"paramLongName": "maxNumberOfRetry",
"paramDescription": "the maximum number of admitted connection retries",
"paramRequired": false
},
{
- "paramName": "rd",
+ "paramName": "rqd",
+ "paramLongName": "requestDelay",
+ "paramDescription": "the delay (ms) between requests",
+ "paramRequired": false
+ },
+ {
+ "paramName": "rtd",
"paramLongName": "retryDelay",
"paramDescription": "the delay (ms) between retries",
"paramRequired": false
},
{
- "paramName": "ct",
+ "paramName": "cto",
"paramLongName": "connectTimeOut",
"paramDescription": "the maximum allowed time (ms) to connect to the remote host",
"paramRequired": false
},
{
- "paramName": "rt",
+ "paramName": "rto",
"paramLongName": "readTimeOut",
"paramDescription": "the maximum allowed time (ms) to receive content from the remote host",
"paramRequired": false
diff --git a/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/collection/oozie_app/workflow.xml b/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/collection/oozie_app/workflow.xml
index e7f6b9201..fe8eea370 100644
--- a/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/collection/oozie_app/workflow.xml
+++ b/dhp-workflows/dhp-aggregation/src/main/resources/eu/dnetlib/dhp/collection/oozie_app/workflow.xml
@@ -95,6 +95,7 @@
--dnetMessageManagerURL${dnetMessageManagerURL}
--mdStoreVersion${wf:actionData('StartTransaction')['mdStoreVersion']}
--maxNumberOfRetry${maxNumberOfRetry}
+ --requestDelay${requestDelay}
--retryDelay${retryDelay}
--connectTimeOut${connectTimeOut}
--readTimeOut${readTimeOut}
diff --git a/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/aggregation/AggregationJobTest.java b/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/aggregation/AggregationJobTest.java
index 3cb66d5ee..ff3ff3b6e 100644
--- a/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/aggregation/AggregationJobTest.java
+++ b/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/aggregation/AggregationJobTest.java
@@ -1,7 +1,7 @@
package eu.dnetlib.dhp.aggregation;
-import static eu.dnetlib.dhp.aggregation.common.AggregationConstants.MDSTORE_DATA_PATH;
+import static eu.dnetlib.dhp.common.Constants.MDSTORE_DATA_PATH;
import static org.junit.jupiter.api.Assertions.assertEquals;
import java.io.File;
diff --git a/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/collector/worker/CollectorWorkerApplicationTests.java b/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/collector/worker/CollectorWorkerApplicationTests.java
index 80bafd6d8..975ef944e 100644
--- a/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/collector/worker/CollectorWorkerApplicationTests.java
+++ b/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/collector/worker/CollectorWorkerApplicationTests.java
@@ -8,9 +8,9 @@ import org.junit.jupiter.api.Test;
import com.fasterxml.jackson.databind.ObjectMapper;
+import eu.dnetlib.dhp.collection.ApiDescriptor;
import eu.dnetlib.dhp.collection.worker.CollectorPluginFactory;
import eu.dnetlib.dhp.collection.worker.HttpClientParams;
-import eu.dnetlib.dhp.collector.worker.model.ApiDescriptor;
@Disabled
public class CollectorWorkerApplicationTests {
diff --git a/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/transformation/TransformationJobTest.java b/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/transformation/TransformationJobTest.java
index 9d6dacf0c..997727e33 100644
--- a/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/transformation/TransformationJobTest.java
+++ b/dhp-workflows/dhp-aggregation/src/test/java/eu/dnetlib/dhp/transformation/TransformationJobTest.java
@@ -1,14 +1,12 @@
package eu.dnetlib.dhp.transformation;
-import static eu.dnetlib.dhp.aggregation.common.AggregationConstants.MDSTORE_DATA_PATH;
+import static eu.dnetlib.dhp.common.Constants.MDSTORE_DATA_PATH;
import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.mockito.Mockito.lenient;
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
-import java.util.Collections;
import java.util.Map;
import java.util.stream.Collectors;
import java.util.stream.Stream;